Skip to content

fix(resident-app): DOMA-13496 prevent double confirmPhoneAction -> Error if sms code was pasted from IOS - #7927

Merged
YEgorLu merged 2 commits into
mainfrom
fix/resident-app/DOMA-13496/prevent-double-confirm-phone-action
Aug 17, 2026
Merged

fix(resident-app): DOMA-13496 prevent double confirmPhoneAction -> Error if sms code was pasted from IOS#7927
YEgorLu merged 2 commits into
mainfrom
fix/resident-app/DOMA-13496/prevent-double-confirm-phone-action

Conversation

@YEgorLu

@YEgorLu YEgorLu commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated the resident app as part of routine maintenance.
    • No user-visible feature or behavior changes were identified in this update.

@YEgorLu YEgorLu added the 👶 small Easy to review changes up to 50 lines of code label Aug 13, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2c41e3d9-fbfa-4e7e-af31-3743c8ade767

📥 Commits

Reviewing files that changed from the base of the PR and between f0187e1 and 58ceaa0.

📒 Files selected for processing (1)
  • apps/resident-app
🚧 Files skipped from review as they are similar to previous changes (1)
  • apps/resident-app

📝 Walkthrough

Walkthrough

The apps/resident-app subproject reference was updated from commit 5259a4e6895421e7fb1f849375802f3528bddc72 to a9d5934164060a2e4319a1488b57da05e2163fb5.

Changes

Resident app update

Layer / File(s) Summary
Update resident app reference
apps/resident-app
The subproject reference now points to commit a9d5934164060a2e4319a1488b57da05e2163fb5.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 58cea

This is a small localized change, and no actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the resident-app fix and the specific double confirmPhoneAction error described in the pull request objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/resident-app/DOMA-13496/prevent-double-confirm-phone-action

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.

@SavelevMatthew SavelevMatthew left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@sonarqubecloud

Copy link
Copy Markdown

@YEgorLu
YEgorLu merged commit 3c364f4 into main Aug 17, 2026
85 of 87 checks passed
@YEgorLu
YEgorLu deleted the fix/resident-app/DOMA-13496/prevent-double-confirm-phone-action branch August 17, 2026 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👶 small Easy to review changes up to 50 lines of code

Development

Successfully merging this pull request may close these issues.

3 participants