Skip to content

Comments

add changes to disable and log reason#7909

Open
Krishnag09 wants to merge 4 commits intodevelopfrom
disable_notif_for_perm_removed
Open

add changes to disable and log reason#7909
Krishnag09 wants to merge 4 commits intodevelopfrom
disable_notif_for_perm_removed

Conversation

@Krishnag09
Copy link
Contributor

Describe briefly what this PR is doing and why.

Testing

Explain how you tested this PR and give clear steps so the reviewer can replicate.

Backend checklist

  • Added tests for any new code or added a regression test if fixing a bug
  • Run the backend locally and it works
  • Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

Web frontend checklist

  • There are no console warnings when running the app
  • Added tests where relevant
  • Clicked around my changes running locally and it works
  • Checked Desktop, Mobile and Tablet screen sizes

For maintainers

  • Maintainers can push commits to my branch
  • Maintainers can merge this PR for me

@Krishnag09 Krishnag09 requested a review from aapeliv as a code owner February 16, 2026 05:15
@vercel
Copy link

vercel bot commented Feb 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
couchers Ready Ready Preview Feb 17, 2026 3:54am

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant