Skip to content
This repository has been archived by the owner on Jul 17, 2020. It is now read-only.

Notifications #375

Open
8 tasks
jspaine opened this issue May 16, 2018 · 4 comments
Open
8 tasks

Notifications #375

jspaine opened this issue May 16, 2018 · 4 comments

Comments

@jspaine
Copy link
Contributor

jspaine commented May 16, 2018

  • Remove notifications when they are viewed
  • Tidy up notifications list
    • Don't show url
    • Format the date nicer
    • Does it need to be a table or would a list be better
  • Other notifications
    • Volunteer created/updated/role changed
    • Donation receipt requested

If anyone can think of other notifications we might want, comment and I'll add them here.

@ronaldblanco
Copy link
Contributor

Hello @jspaine
I will begin to work on this steps.

@ronaldblanco
Copy link
Contributor

Hello @jspaine
Hey I am sorry I have been a little sort of time but I have been checking (to resolve the 1 point "Remove notifications when they are viewed") but I do not fine in the application other side were sometime like that it is working. Could you give me and AB of how to start this??

Thanks.

@jspaine
Copy link
Contributor Author

jspaine commented Jun 14, 2018

Hi, no problem. I think it's just to make it so when you click to view a notification you dispatch an action from the deleteNotification action creator before navigating.

The api call in deleteNotification should probably have schema: null, responseSchema: arrayOfNotifications though, because you're only sending a notification id and getting back the new list of notifications.

@ronaldblanco
Copy link
Contributor

Hey @jspaine take a look and let me know.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants