Skip to content

Migrate existing data and cut over to /api/v2 (#141) - #162

Merged
MatheusCastro99 merged 4 commits into
masterfrom
feat/141-v2-migration
Nov 14, 2026
Merged

MatheusCastro99 merged 4 commits into
masterfrom
feat/141-v2-migration

Conversation

@MatheusCastro99

Copy link
Copy Markdown
Owner

Closes #141. Plan: #141 (comment)

🤖 Generated with Claude Code

MatheusCastro99 and others added 3 commits November 13, 2026 15:59
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…, restore keeps query options (#141)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@MatheusCastro99
MatheusCastro99 marked this pull request as ready for review November 14, 2026 03:55
@MatheusCastro99

Copy link
Copy Markdown
Owner Author

G2: ready to merge at 7f6c6acf088e55b7ebac33f19c3769b80ec5e646. Reviewed SHA = PR head = the SHA CI passed on.

  • G2 at 2ccea2e: the red-team found nothing blocking. The UI check passed on migrated throwaway data: dates, recomputed totals, the PDF page, and every request goes to /api/v2/.
  • The owner approved fixes 1–4: the dry run without --, no collection created by a dry run, error codes in failure messages, and the restore keeping the query options. They are in 7f6c6ac, and the narrow re-review approved them.
  • Findings 5 and 6 (existing frontend behaviour) were added to backlog Backlog: bugs found during epic work #153.

Before merge (acceptance criterion, owner only): run the README procedure ("Migrating a database from before v2.0") on your own database, and fix anything it reports.

Note from the re-review (non-blocking, no change made): the restore placeholder in Docs/README.md doesn't say to keep the / before ?. Someone who deletes /<database> entirely gets mongodb://host?authSource=..., and the connection-string format needs mongodb://host/?authSource=... (not confirmed with mongorestore itself). If that happens, it fails before --drop runs, so no data is at risk. Recommendation: a one-line wording fix, here before merge or at /close.

The owner merges, then runs /close 141.

Copilot AI balanced review requested due to automatic review settings November 14, 2026 03:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@MatheusCastro99
MatheusCastro99 merged commit 65ecfd0 into master Nov 14, 2026
8 checks passed
@MatheusCastro99
MatheusCastro99 deleted the feat/141-v2-migration branch November 14, 2026 12:44
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.

Migrate existing data and cut over to /api/v2

2 participants