Skip to content

:chore: Fix the masking boxes appearances during proof generation#18

Merged
wryonik merged 1 commit intomainfrom
shubham/reg-637-fix-the-masking-boxes-appearance-during-proof-generation
Dec 29, 2025
Merged

:chore: Fix the masking boxes appearances during proof generation#18
wryonik merged 1 commit intomainfrom
shubham/reg-637-fix-the-masking-boxes-appearance-during-proof-generation

Conversation

@wryonik
Copy link
Copy Markdown
Collaborator

@wryonik wryonik commented Dec 23, 2025

No description provided.

@linear
Copy link
Copy Markdown

linear Bot commented Dec 23, 2025

REG-637

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 23, 2025

Walkthrough

The useBlackMask prop condition in the EmailCard component was changed from checking proof generation state (generatedProof !== null) to checking verification URL availability (verificationUrl !== null), altering the trigger for mask display behavior.

Changes

Cohort / File(s) Summary
EmailCard prop condition update
src/App.tsx
Modified useBlackMask prop condition from generatedProof !== null to verificationUrl !== null

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Free

📥 Commits

Reviewing files that changed from the base of the PR and between 945774f and 03de622.

📒 Files selected for processing (1)
  • src/App.tsx

Note

🎁 Summarized by CodeRabbit Free

The PR author is not assigned a seat. To perform a comprehensive line-by-line review, please assign a seat to the pull request author through the subscription management page by visiting https://app.coderabbit.ai/login.

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

@wryonik wryonik requested a review from BornPsych December 23, 2025 14:12
@wryonik wryonik merged commit c9ff2b8 into main Dec 29, 2025
3 checks passed
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