Skip to content

chore(notifications): rename approval notifications to nudges in the UI #56505

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

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

cathteng
Copy link
Member

Screenshot 2023-09-19 at 14 09 58 Screenshot 2023-09-19 at 14 09 40

The url is still /settings/account/notifications/approval/ though.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Sep 19, 2023
@cathteng cathteng requested a review from scefali September 19, 2023 21:17
@cathteng cathteng marked this pull request as ready for review September 20, 2023 16:20
@@ -59,14 +59,14 @@ export const NOTIFICATION_SETTING_FIELDS: Record<string, Field> = {
},
},
approval: {
name: 'approval',
name: 'Nudges',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cathteng let's not change the name here but just change the label

@cathteng cathteng enabled auto-merge (squash) September 20, 2023 16:34
@cathteng cathteng merged commit 46b3e77 into master Sep 20, 2023
@cathteng cathteng deleted the cathy/notifications/rename-approvals-to-nudges branch September 20, 2023 16:50
@github-actions github-actions bot locked and limited conversation to collaborators Oct 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants