Skip to content

Conversation

@transphorm
Copy link
Member

@transphorm transphorm commented Jan 10, 2026

Description

A brief description of the changes, what and how is being changed.

Tested

Explain how the change has been tested (for example by manual testing, unit tests etc) or why it's not necessary (for example version bump).

How to QA

How can the change be tested in a repeatable manner?

Summary by CodeRabbit

  • Refactor
    • Modified verification screens to no longer automatically navigate when disclosure data becomes stale.

✏️ Tip: You can customize this high-level summary in your review settings.

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 10, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR removes the useProofDisclosureStalenessCheck hook from two verification screens, eliminating automatic staleness validation and navigation logic that previously redirected to Home when disclosure data was stale.

Changes

Cohort / File(s) Summary
Verification Screen Hook Removal
app/src/screens/verification/DocumentSelectorForProvingScreen.tsx, app/src/screens/verification/ProveScreen.tsx
Deleted imports and invocations of useProofDisclosureStalenessCheck. Removes automatic runtime staleness checks that navigated to Home on stale disclosures. Control flow simplified by eliminating this hook dependency.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

codex

Suggested reviewers

  • remicolin

Poem

🧹 A staleness check fades away,
Two screens shed their safety sway,
Simpler paths through prove and play,
Navigation flows the modern way! ✨

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c25a35e and 82394e1.

📒 Files selected for processing (2)
  • app/src/screens/verification/DocumentSelectorForProvingScreen.tsx
  • app/src/screens/verification/ProveScreen.tsx

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@transphorm transphorm merged commit 80d2181 into dev Jan 10, 2026
16 of 17 checks passed
@transphorm transphorm deleted the justin/remove-proving-staleness-check-for-now branch January 10, 2026 15:39
jcortejoso pushed a commit that referenced this pull request Jan 15, 2026
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.

2 participants