Giveth / notification-center

It's a microservice that would handle all notification of giveth.io and trace.giveth.io
GNU General Public License v3.0
24 stars 2 forks source link

Notification Center - Notifcations on status change of projects #34

Closed laurenluz closed 1 year ago

laurenluz commented 1 year ago

@MoeNick - as per your request, here is some copy/flow for various project status changes. Can you please review them and then assign them to the correct person to implement?

@MoeNick - I need your confirmation on these ones because I'm not 100% certain I am correct here:

MoeNick commented 1 year ago

Thanks @laurenluz Actually I meant copies for supported projects for above events

laurenluz commented 1 year ago

like notifcations to sent to people who "supported" projects when the project they supported has a status change? @MoeNick

MoeNick commented 1 year ago

yes

laurenluz commented 1 year ago

Notifications to "supported" when the project they supported has a status update, or provides an update:

For settings page (I think it is ok as it is): image

@MoeNick - is this good now?

MoeNick commented 1 year ago

Thanks a lot @laurenluz

MoeNick commented 1 year ago

Shall I move it to done or I have to ask @alireza-sharifpour to do something to get ready to test?

mohammadranjbarz commented 1 year ago

Shall I move it to done or I have to ask @alireza-sharifpour to do something to get ready to test?

I think it's better to wait for frontend guys and after that we can tell @maryjaf to test all these items

alireza-sharifpour commented 1 year ago

Shall I move it to done or I have to ask @alireza-sharifpour to do something to get ready to test?

I'm currently working on PFP collection, after that i'll work on it.

maryjaf commented 1 year ago
  • [x] Project deactivated --> "< project name >, which you supported, has been deactivated by the project owner."
  • [x] Project activated --> "< project name >, which you supported, has been reactivated by the project owner."

It is possible to deactivate and activate the project by both the project owner and the admin user Should the same message be displayed in both modes? @MoeNick @laurenluz

maryjaf commented 1 year ago

image.png This message needs to be corrected (Unlisted status for. project owner)

mohammadranjbarz commented 1 year ago

image.png This message needs to be corrected (Unlisted status for. project owner) @maryjaf Thank you I fixed it

Screen Shot 1401-11-23 at 11 40 13
MoeNick commented 1 year ago
  • [x] Project deactivated --> "< project name >, which you supported, has been deactivated by the project owner."
  • [x] Project activated --> "< project name >, which you supported, has been reactivated by the project owner."

It is possible to deactivate and activate the project by both the project owner and the admin user Should the same message be displayed in both modes? @MoeNick @laurenluz

Yes let's remove "by .... " [ ] Project deactivated --> "< project name >, which you supported, has been deactivated." [ ] Project activated --> "< project name >, which you supported, has been reactivated . @mohammadranjbarz ؛ث

mohammadranjbarz commented 1 year ago
  • [x] Project deactivated --> "< project name >, which you supported, has been deactivated by the project owner."
  • [x] Project activated --> "< project name >, which you supported, has been reactivated by the project owner."

It is possible to deactivate and activate the project by both the project owner and the admin user Should the same message be displayed in both modes? @MoeNick @laurenluz

Yes let's remove "by .... " [ ] Project deactivated --> "< project name >, which you supported, has been deactivated." [ ] Project activated --> "< project name >, which you supported, has been reactivated . @mohammadranjbarz ؛ث

@maryjaf @MoeNick Done

maryjaf commented 1 year ago

In some sentences, there is no end point, plz change this @mohammadranjbarz image.png

maryjaf commented 1 year ago
  • Project deactivated --> "You have deactivated your project < project name >."

I think since it is currently possible to disable project both by admin and by project owner, it would be better to change this sentence. @MoeNick @laurenluz

mohammadranjbarz commented 1 year ago

In some sentences, there is no end point, plz change this @mohammadranjbarz image.png

@maryjaf Done.