Contrib.social

Make behavior of alerts on translated pages configurable

Open on Drupal.org →
Created on 22 September 2023, 9 months ago
Updated 14 May 2024, about 1 month ago

Problem/Motivation

The issue ✨ Respect selected language for alert Fixed proposed that alerts only show on pages that match the alert language, but there are also use cases where it is desirable for translated pages to display an untranslated alert.

For example, a page for a public park needs an alert because of a wildfire, zombie apocalypse or other life-threatening emergency. The alert needs to go on the site, but there isn't time to have the content translated. It should be possible to display the alert in the original language on translated pages.

Proposed resolution

Add a configuration option to fall back on the original language, set to false by default. This way, the current behavior is retained when the module is updated.

In SitewideAlertManager::activeSitewideAlerts, check the configuration before filtering the collection on the current language.

✨ Feature request
Status

Needs work

Version

2.0

Component

Code

Created by

🇺🇸United States calebtr

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @calebtr
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update 9 months ago
    8 pass
  • @calebtr opened merge request.
  • Status changed to Needs review 9 months ago
  • Comment 9 months ago →
    🇺🇸United States ChrisSnyder Maryland
  • Status changed to Needs work 4 months ago
  • Comment 4 months ago →
    🇳🇱Netherlands Arantxio Dordrecht

    I've tested this and it seems to be working, but i think it would be better if we add a option on the specific alerts once this is enabled. Now it will always show untranslated items, even if you don't want it to.

  • Comment about 1 month ago →
    🇺🇸United States calebtr

    If that level of customization is needed, it might be better to replace the query with a view. Is there room in the roadmap for that?

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024