-
Notifications
You must be signed in to change notification settings - Fork 816
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
Dismiss notifications #5687
Dismiss notifications #5687
Conversation
fa333ef
to
dcd7783
Compare
dcd7783
to
3b9cc7c
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #5687 +/- ##
==========================================
- Coverage 59.28% 59.27% -0.01%
==========================================
Files 143 143
Lines 18463 18463
==========================================
- Hits 10945 10944 -1
- Misses 7518 7519 +1 |
3b9cc7c
to
dae575e
Compare
dae575e
to
3ab72ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see my question on the comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, though Matthieu's comment would be good to fix
3ab72ab
to
0931c1b
Compare
- Fix for #5606 and #5585. - That is how it is done in the server and other clients. - Update tests. Signed-off-by: Camila <[email protected]>
0931c1b
to
d6db8fc
Compare
Kudos, SonarCloud Quality Gate passed! |
AppImage file: nextcloud-PR-5687-d6db8fcbd6247909feb3fb7508a56f5d2ce36562-x86_64.AppImage |
Any notification missing a dismiss link will get one like in the server: