Skip to content

fix(settings): invalidate cache in notification settings#112161

Merged
natemoo-re merged 4 commits intomasterfrom
settings/invalidate-cache
Apr 3, 2026
Merged

fix(settings): invalidate cache in notification settings#112161
natemoo-re merged 4 commits intomasterfrom
settings/invalidate-cache

Conversation

@natemoo-re
Copy link
Copy Markdown
Member

Some notification settings were not updating the underlying data on success, leading to the form being reset with stale initialData. In effect, the form appeared to not trigger any update from the frontend—a hard refresh would display the correct values because the backend did process the update.

Closes DE-1043, closes #112143

@natemoo-re natemoo-re requested a review from a team as a code owner April 2, 2026 20:50
@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 2, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 2, 2026
Comment thread static/app/views/settings/account/notifications/notificationSettings.tsx Outdated
natemoo-re and others added 2 commits April 3, 2026 09:06
@natemoo-re natemoo-re enabled auto-merge (squash) April 3, 2026 13:06
@natemoo-re natemoo-re merged commit bef39b9 into master Apr 3, 2026
74 checks passed
@natemoo-re natemoo-re deleted the settings/invalidate-cache branch April 3, 2026 13:15
@github-actions github-actions bot locked and limited conversation to collaborators Apr 19, 2026
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.

Person User Notification Updates Not Reflected on Page Until Manual Refresh

2 participants