Carry the remaining attempts in the deletion code error - #499
Conversation
|
Approach:
This shape keeps the stable error codes and response contracts intact. I am using the central message catalog instead of endpoint-specific strings, and the named constant instead of a second duration literal, so policy and copy cannot drift. |
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes I reviewed the setup commit currently on PR #499.
- Empty setup commit No files or runtime behavior differ from
main; the planned error-message and test updates are not present yet and will need an incremental review after they land.
GPT Sol | 𝕏
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes I reviewed the implementation added since the prior Pullfrog review.
- Deletion attempt feedback Formatted account-deletion verification failures with the post-failure remaining-attempt count, matching the established API-key challenge behavior.
- Exhausted-attempt guidance Replaced the shared request-a-new-code fallback with a wait message derived from
VerificationAttemptWindowMinutes. - Cross-repo alignment Confirmed paired orbit-ui-mobile PR #774 consumes the exact remaining-attempt suffix for deletion and API-key step-up screens.
- Behavioral coverage Tightened application tests around each deletion attempt and the exhausted copy across all consumers; the solution build and all 5,980 tests passed.
GPT Sol | 𝕏
|
|
@pullfrog review |
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes Reviewed the deletion challenge error copy, shared exhausted-attempt guidance, generated architecture refresh, and focused regression coverage against paired UI PR #774.
-
Remaining attempts: Formats the deletion invalid-code error with the server-side count consumed by the web and mobile step-up flow.
-
Exhaustion guidance: Uses the shared 15-minute constant, matching the absolute attempt-cache expiration across deletion, API-key creation, and login verification.
-
Regression coverage: Tightens expectations across deletion, API-key, and auth verification flows. The focused application test run passed all 46 tests.
-
Architecture artifacts: Refreshes the generated application file count after the source change.
GPT Sol | 𝕏




Pairs with UI ticket #337: https://github.com/thomasluizon/orbit-tickets/issues/337
This changes the deletion wrong-code message to include the remaining attempt count. It also changes exhausted-attempts copy to tell the person to try again in 15 minutes instead of requesting a new code.
No response fields or endpoint shapes change.