citizenos / citizenos-fe

5 stars 2 forks source link

IDEATION: Add in ideation to notifications text #1177

Closed BeccaMelhuish closed 17 hours ago

BeccaMelhuish commented 4 months ago

@ilmartyrk please add in text for whatever ideation notifications there will be :)

Maybe you can let me know what notifications get sent for (ideas, replies, 'upvoted' ideas, new folders, favourited, deleted ideas, etc? not sure it should be for all of them?) and then I can check the text?

image

BeccaMelhuish commented 4 months ago

@ilmartyrk checked but not seeing it? :)

image

BeccaMelhuish commented 1 month ago

@ilmartyrk I've re-opened again as still don't see it in the list :)

image

There should be a row saying:

Idea gathering (ideas, replies, reactions)

Or, if it's not separated out from discussion notifications, then we'd need to change the 'Topic discussion' row copy to:

Topic contributions (ideas, arguments, replies, reactions)

One or the other, whichever is relevant to how it works :)

ilmartyrk commented 17 hours ago

Added Ideation as a separate item in the list for translations, technical side will still need some work as the whole notification system needs some work

ssin1901 commented 12 hours ago

@BeccaMelhuish @ilmartyrk I guess this would need to be tested in the test environment?

BeccaMelhuish commented 11 hours ago

@ssin1901 I'm also getting confused which place to test at the moment so am just checking both if I don't find it in test :D

This one is showing in test though: image

But @ilmartyrk, just to check what did you mean about the technical side - is that this issue, or something else? :)