Contrib.social
Created on 13 October 2021, over 3 years ago
Updated 30 March 2023, about 2 years ago

Problem/Motivation

Just getting up to speed on this module and interested in a few open issues:
- #3243392: Allow Sitewide Alerts to be Fieldable → (mine)
- #3229142: Add ability to put alerts in blocks →

and i started thinking about the idea of "alert types". Consider this:

I would suggest splitting existing/new functionality across bundle/entity as:

Bundle/Type:
- option to use default placement at top of page or create a block for that Type
- define page visibility per type - this gives you the ability to define a type to be placed in a certain location on certain pages - technically the page visibility isnt required any more as block setting can handle that
- this way visibility doesnt have to be redefined each time an alert is added - it is simply an alert of Type X

Entity/Alerts:
- still off scheduling and style (type would be a better word than style; but i admit that will get confusing once bundles are added).

I havent yet stumbled upon how to make alert entities fieldable without providing bundles (as is done with User entity); but i do know how to do this if we add ability to add bundles - so that solves my fieldable issue as well. :)

Just some thoughts. The fieldable addition i need for sure but assuming that can be done without adding bundles (at least with a fixed single bundle). But if we also need "site section" alerts on top of site wide alerts.. then this will mean different placement and therefore blocks - and then i think a very good reason to have alert types. I can do the code for all of this if maintainer is interested in this feature enhancement.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

✨ Feature request
Status

Postponed: needs info

Version

2.0

Component

Code

Created by

🇨🇦Canada liquidcms

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Comment about 2 years ago →
    🇧🇪Belgium beerendlauwers

    Changing the "Alert Type" dropdown to become the bundle type seems like a great way to migrate existing alerts and align the structure of this module to Drupal's entity type architecture.

  • Status changed to Active 6 months ago
  • Comment 6 months ago →
    🇺🇸United States smustgrave
  • Comment 7 days ago →
    🇺🇸United States swirt Florida

    - technically the page visibility isnt required any more as block setting can handle that

    I don't agree with this. There is a big difference between the block level page targeting and the individual alert level page targetting. In the several places I am using this module, we need specific alerts to show on specific paged even if they are all the same type of alert.

    I do like the idea of having types, but then you also need multiple blocks for being able to place different types in different regions.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024