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.

Sign up for GitHub

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

Jump to bottom

MVP Notification #1010

Merged
merged 19 commits into from
Oct 2, 2023
Merged

MVP Notification #1010

merged 19 commits into from
Oct 2, 2023

Conversation

caneppelevitor
Copy link
Collaborator

@caneppelevitor caneppelevitor commented Sep 15, 2023 •
edited by pepermao
Loading

Icon when it has notifications

image

Notification Menu

image

Unread notifications

image

@pepermao pepermao changed the title Notification Feature MVP Notification Sep 19, 2023
@pepermao pepermao marked this pull request as ready for review September 19, 2023 22:52
@pepermao pepermao force-pushed the notification-feature branch 2 times, most recently from 5bfd1d5 to 001a49b Compare September 30, 2023 21:47
caneppelevitor and others added 17 commits October 2, 2023 18:33
@caneppelevitor @pepermao
Novu installation
ef7bcc6
@caneppelevitor @pepermao
Start notification service and provider
79cb0a7
@pepermao
- create notification module
d76696b 
- create subscription function
@caneppelevitor @pepermao
-Notification Controller
05f077d 
-Notification Api
@caneppelevitor @pepermao
-Creation of:
2496579 
-Topic creation endPoint
-Add Subscriber to Topic endPoint
-Send Topic Notification endPoint
-Service Functions for each
@caneppelevitor @pepermao
Changed endPoints names
d01aeb5
@caneppelevitor @pepermao
-Refactor createSubscriber function
743d9be 
-Creatiton of functions for remove a user from a topic and verify if a user is subscribed in a topic
@pepermao
- create notification components
01b27c6 
- improve notificaiton interface
- send notification on every machine states
- fixed user register
@thesocialdev @pepermao
Fix jest tests by updating libraries and fixing jest preset
450d400
@thesocialdev @pepermao
Get NOVU_API_KEY from the config yaml
34cf6ff
@pepermao
- implement hmac encryption
b6a8b9f
@pepermao
- fixed cypress test
7ebb8dc 
- prevent notification token does not provide behavior
@pepermao
- fixed tests by downgrading axios version
72ed06a
@pepermao
create novu subscribers mongodb migration
84aedb7
@pepermao
fixed yarn tests by adding a if statement on every novu function to c…
39489c4 
…heck if it is configured

- change ory requests from axios to node fetch
- update packages
@pepermao
- check if user has session before show notification icon
9945cde
@pepermao
- forgot to add header content file
abe96dc
@sonarcloud
Copy link

sonarcloud bot commented Oct 2, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@thesocialdev thesocialdev merged commit 89bcfcc into stage Oct 2, 2023
4 checks passed
@thesocialdev thesocialdev mentioned this pull request Oct 4, 2023
Merged
@thesocialdev thesocialdev deleted the notification-feature branch August 3, 2024 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pepermao pepermao pepermao left review comments

@thesocialdev thesocialdev thesocialdev approved these changes

@giovannirossini giovannirossini Awaiting requested review from giovannirossini giovannirossini is a code owner

Assignees

@pepermao pepermao

@caneppelevitor caneppelevitor

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@caneppelevitor @thesocialdev @pepermao