Skip to content

Restore original notes when a week is reset - #30

Merged
Drefvelin merged 1 commit into
mainfrom
restore-patch-notes
Sep 24, 2026
Merged

Drefvelin merged 1 commit into
mainfrom
restore-patch-notes

Conversation

@Drefvelin

@Drefvelin Drefvelin commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Reset drops lines that came from a deny reason and puts the original notes back to pending.
  • Generated notes are left in place. Release keys are not cleared.

Test plan

  • pytest src/api/test_patchnotes_routes.py
  • /patchnotes reset removes the feedback line and brings back the original note

Made with Cursor

Summary by CodeRabbit

  • Bug Fixes
    • Resetting a week now preserves its original notes, removes revision notes, and restores denied notes to pending review instead of clearing all notes.
    • Reset results now report separate counts for removed revisions and restored notes.

Deny edits are dropped. The generated lines stay.

Co-authored-by: Cursor <[email protected]>
@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: d70749de-815e-4284-a443-0eabe84c7844

📥 Commits

Reviewing files that changed from the base of the PR and between aa5e4fd and f9d580f.

📒 Files selected for processing (3)
  • backend/src/api/patchnotes_routes.py
  • backend/src/api/test_patchnotes_routes.py
  • backend/src/patchnotes/db.py
 ________________________________________________
< Press W-A-S-D to move me around your codebase. >
 ------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@Drefvelin
Drefvelin merged commit 19c0445 into main Sep 24, 2026
1 of 2 checks passed
@Drefvelin
Drefvelin deleted the restore-patch-notes branch September 24, 2026 09:42
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