Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SOLID notifications #142

Open
3 of 4 tasks
fernandossmm opened this issue Apr 6, 2020 · 2 comments
Open
3 of 4 tasks

SOLID notifications #142

fernandossmm opened this issue Apr 6, 2020 · 2 comments
Assignees
Labels
Back-end Something that has to do with Back-end development enhancement New feature or request utility Utilities and other useful features for development

Comments

@fernandossmm
Copy link
Contributor

fernandossmm commented Apr 6, 2020

Feature description

The app should be able to manage the notification system, it should provide SOLID users a way of being notified when a route is shared with them or when a comment has been made on their own route.

Acceptance criteria

  • Way of posting a notification of type 'routeShared'.

  • Way of posting a notification of type 'commentMade'.

  • Way of retrieving and processing the url of the route of a notification type 'routeShared'.

  • Way of retrieving the notifications made.

@fernandossmm fernandossmm added enhancement New feature or request utility Utilities and other useful features for development Back-end Something that has to do with Back-end development labels Apr 6, 2020
@ClaraMirandaGarcia ClaraMirandaGarcia self-assigned this Apr 10, 2020
@ClaraMirandaGarcia
Copy link
Contributor

A class NotificationManager.js was created with the following functions:

  • createNotificationContent: method that allows to create the content of the notification itself.

  • postNotification: being the method in charge to send the notification to the inbox of the friend.

  • getNotificationDocuments: it retrieves the documents of the notifications placed in the inbox folder

  • processSharedRoutes: it allows to retrieve the 'summary' of the notification, where the url of the route shared is contained.

This approach using tripledoc was taken when the hook useNotification did not work properly.

@ClaraMirandaGarcia
Copy link
Contributor

ClaraMirandaGarcia commented Apr 23, 2020

With the postNotification method there was a problem, in order to fix it, I turned to the solid community, and so, it was described and solved in this post

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Back-end Something that has to do with Back-end development enhancement New feature or request utility Utilities and other useful features for development
Projects
None yet
Development

No branches or pull requests

2 participants