Skip to content

Improve GitHub re-verify flow with cooldown, clearer errors, and UI feedback - #297

Merged
admdly merged 2 commits into
mainfrom
codex/update-error-handling-in-developer-reverify
Aug 1, 2026
Merged

Improve GitHub re-verify flow with cooldown, clearer errors, and UI feedback#297
admdly merged 2 commits into
mainfrom
codex/update-error-handling-in-developer-reverify

Conversation

@admdly

@admdly admdly commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Description

  • Enhanced server-side error handling in POST /account/developer/reverify to inspect ApiRequestError code values and map them to specific user-facing descriptions, including handling RATE_LIMITED, SERVICE_UNAVAILABLE, and GITHUB_ENTITY_UNSUPPORTED cases.
  • Store a short cooldown timestamp in the session under reverifyCooldownUntil when rate-limited or service-unavailable responses are returned so the UI can temporarily disable reverify attempts.
  • Updated account page (src/pages/account/index.astro) to read and clear reverifyCooldownUntil from the session and pass it to the reverify form as data-cooldown-until and data-reverify-form.
  • Disabled the reverify button when a cooldown is active and update its label to Try again shortly while disabled, plus added a client-side script to manage button state, show a Re-verifying… label and aria-busy while submitting, and re-enable the button when the cooldown expires.
  • Adjusted flash messages to include a consistent title (Could not refresh GitHub verification) and a more specific description field when errors occur.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 1, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
extensions df872e1 Commit Preview URL

Branch Preview URL
Aug 01 2026, 11:21 PM

@admdly
admdly marked this pull request as ready for review August 1, 2026 23:01
@admdly admdly self-assigned this Aug 1, 2026
@admdly

admdly commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9293bfaae6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/pages/account/index.astro
@admdly

admdly commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

@cubic review

@admdly
admdly merged commit bb2ca42 into main Aug 1, 2026
8 checks passed
@admdly
admdly deleted the codex/update-error-handling-in-developer-reverify branch August 1, 2026 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant