issues
search
coralproject
/
talk
A better commenting experience from Vox Media
https://coralproject.net
Other
1.89k
stars
354
forks
source link
DSA no notifications placeholder text
#4408
Closed
nick-funk
closed
11 months ago
nick-funk
commented
11 months ago
What does this PR do?
Show placeholder text when user has no notifications
Make method of redress email/url a link
These changes will impact:
[X] commenters
[ ] moderators
[ ] admins
[ ] developers
What changes to the GraphQL/Database Schema does this PR introduce?
None
Does this PR introduce any new environment variables or feature flags?
No
If any indexes were added, were they added to
INDEXES.md
?
No new indices
How do I test this PR?
Turn on DSA mode in Admin > Configure > General > DSA
Select the Email or URL option for method of redress and provide a value
Make a new user
Check their notifications
See that they have a placeholder when they have no notifications
Create a comment
Reject it
Check notifications
See that the notification appeared for the rejection
See that the method of redress shows with a link to the email/url option you filled out in the config earlier
Where any tests migrated to React Testing Library?
No
How do we deploy this PR?
Merge to DSA epic branch
netlify[bot]
commented
11 months ago
✅
Deploy Preview for
gallant-galileo-14878c
canceled.
Name
Link
🔨
Latest commit
b6bdc40736f2063a496aa8424bff9f0e3e58a3c2
🔍
Latest deploy log
https://app.netlify.com/sites/gallant-galileo-14878c/deploys/655b902da61bc800088e86be
What does this PR do?
These changes will impact:
What changes to the GraphQL/Database Schema does this PR introduce?
None
Does this PR introduce any new environment variables or feature flags?
No
If any indexes were added, were they added to
INDEXES.md
?No new indices
How do I test this PR?
Where any tests migrated to React Testing Library?
No
How do we deploy this PR?