Skip to content

✨ app: scan pix qr codes - #1288

Open
nfmelendez wants to merge 2 commits into
mainfrom
nicolas
Open

✨ app: scan pix qr codes#1288
nfmelendez wants to merge 2 commits into
mainfrom
nicolas

Conversation

@nfmelendez

Copy link
Copy Markdown
Contributor

No description provided.

@nfmelendez

Copy link
Copy Markdown
Contributor Author

@codex review

@changeset-bot

changeset-bot Bot commented Sep 3, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9138bd0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@exactly/server Patch
@exactly/mobile Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 57 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Team

Run ID: d7e6c161-9f9f-43af-8274-49d957e88ba1

📥 Commits

Reviewing files that changed from the base of the PR and between be72b1a and 9138bd0.

📒 Files selected for processing (15)
  • .changeset/brave-otters-shake.md
  • .changeset/silver-falcons-stick.md
  • cspell.json
  • server/test/utils/bridge.test.ts
  • server/utils/ramps/bridge.ts
  • src/components/send-funds/NewRecipient.tsx
  • src/components/send-funds/QR.tsx
  • src/components/send-funds/Recipients.tsx
  • src/components/send-funds/Scanner.tsx
  • src/components/send-funds/SendAmount.tsx
  • src/components/send-funds/recipientForm.tsx
  • src/i18n/es-AR.json
  • src/i18n/es.json
  • src/i18n/pt.json
  • src/utils/pix.ts

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.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: 9138bd0051

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.37%. Comparing base (be72b1a) to head (9138bd0).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
server/utils/ramps/bridge.ts 95.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1288      +/-   ##
==========================================
+ Coverage   72.51%   73.37%   +0.85%     
==========================================
  Files         275      276       +1     
  Lines       13256    13445     +189     
  Branches     4610     4702      +92     
==========================================
+ Hits         9613     9865     +252     
+ Misses       3306     3241      -65     
- Partials      337      339       +2     
Flag Coverage Δ
e2e 72.45% <70.00%> (-0.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

3 participants