Skip to content

Fix/conflicts #1178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 14 commits into from
Sep 3, 2025
Merged

Fix/conflicts #1178

merged 14 commits into from
Sep 3, 2025

Conversation

kushagrasarathe
Copy link
Contributor

No description provided.

Zishan-7 and others added 14 commits August 14, 2025 18:11
@Zishan-7
HOTFIX - IBAN country detection and incorrect bank acc details (#1094)
820902b 
* Fix: Iban country detection and incorrect bank acc details

* Fix: update IBAN country validation to use correct locale string comparison

* add validations for US and mexican bank accounts

* fix typo

* fix claim flow and create a reusable function for getting 3 letter code

* fix country code mismatch

* fix: show error below input field

* remove unnecessary checks

* remove unnecessary CLABE check
@Zishan-7
Prod LP v2.1 (#1098)
ae3ac97 
* feat: lpv2.1

* fix: gigaclouds, font and exchange widget

* fixes and improvements

* remove duplicate export

* remove unused component
@Zishan-7
Fix: Landing page hero section responsiveness issue (#1107)
2f6a201 
* fix: hero section responsiveness issue

* fix: stars position

* fix height on desktop

* remove unused code
@Zishan-7
fix margins (#1113)
da2c464
@jjramirezn @kushagrasarathe
@Zishan-7 @Hugo0
[TASK-14052] Prod release 105 (#1122)
e64e47a 
* feat: handle send link claims to bank account for peanut users (#1078)

* reafactor: create reusable country list component and use it for all the flows

* feat: reusable user accounts components

* feat: handle different cases based on kyc status for bank claim

* fix: account creation

* chore: add docstring to hooks

* chore: better comments for bank flow manager

* fix: kyc modal closing after tos acceptance issue

* fix: remove bank acc caching from withdraw flow

* fix: update confirm claim modal copy

* fix: remove bank acc caching from claim flow

* fix: navheader title

* remove duplicate debounce code and use `useDebounce` hook instead (#1079)

* Landing page v2.1 (#1089)

* lpv2.1 part 1

* Add exchange widget

* add and integrate exchange API

* add yourMoney component bg

* update landing countries svg

* integrate frankfurter API

* fixes and improvements

* decrease hero section height

* allow max 2 decimal places

* Add `/exchange` route

* fix: overlay

* make destination amount editable and bugg fixes

* some fixes & currency improvements

* crucial commit

* fix checkmark, font size and weight

---------

Co-authored-by: Hugo Montenegro <[email protected]>

* [TASK-13186] refactor: use networkName instead of axelarChainName (#1095)

* refactor: use networkName instead of axelarChainName

* fix: types

* fix: onramp currency (#1096)

* fix: stretched favicon (#1099)

* [TASK-13971] fix: scientific notation in eip681 parsing (#1097)

* fix: scientific notation in eip681 parsing

* fix: qr handling tests

* fix: peanut sdk mock

* pull iban hotfix (#1100)

* fix: claim flow bugs (#1102)

* fix: cross chain claim

* fix: full name issue on confirm bank claim view

* fix: back navigation on desktop views

* Fix back button not working on /profile (#1101)

* Fix back button not working

* fix public profile page

* extract internal navigation logic to utility function

* fix: send link claims to us bank accounts (#1108)

* fix: usa bank account claims

* fix: show bank account details in confirm claim view

* Sync Landing page changes (#1111)

* reduce clouds size and update font

* fix: hero section responsiveness issue

* fix: formatting errors

* add currency animation

* fix: us bank claims after kyc for logged in users (#1112)

* fix: trim account form inputs for spaces (#1114)

* [TASK-14107] fix: don't allow claiming on xChain if route is not found (#1115)

* fix: don't allow claiming on xChain if route is not found

* fix(claim): use correct decimals for min receive amount

* feat: handle redirect uri when on unsupported browsers (#1117)

* feat: handle redirect uri when on unsupported browsers

* fix: confirm bank claim ui rows for iban guest claim

* remove animation (#1118)

* fix: formatting

---------

Co-authored-by: Kushagra Sarathe <[email protected]>
Co-authored-by: Mohd Zishan <[email protected]>
Co-authored-by: Hugo Montenegro <[email protected]>
@kushagrasarathe
fix: bank claim flow runtime error (#1138)
874f15b
@kushagrasarathe
hotfix: make iban non optional (#1139)
bf603ed 
* fix: bank claim flow runtime error

* fix: dont have iban as optional

* fix: merge conflicts
@jjramirezn
fix: merge external account with bank details (#1140)
fd5b66b
@jjramirezn
fix: add id to external account (#1142)
74109fb
@Hugo0
added tg footer (#1144)
4770c38
@Zishan-7
Hotfix : add missing countries - claim as guest flow (#1146)
1b35b32 
* fix: add missing countries

* remove duplicate comment
@jjramirezn
fix: show error on dynamic bank account form (#1147)
04e2142
@Zishan-7
fix: Invalid IBAN for UK (#1151)
70c2a03
@kushagrasarathe
Merge branch 'peanut-wallet' into peanut-wallet-dev
ece694d
@coderabbitai coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 3, 2025 •
edited
Loading

Caution

Review failed

The pull request is closed.

Walkthrough

Formatting-only updates: standardized export spacing/quotes in illustrations index and adjusted imports in LandingPage Footer (reordered asset imports, removed explicit React import for JSX runtime). No logic or public API changes.

Changes

Cohort / File(s) Summary
Assets exports formatting
src/assets/illustrations/index.ts		 Reformatted ThinkingPeanut export: adjusted spacing and switched to single quotes; no semantic change.
Footer import cleanup
src/components/LandingPage/Footer.tsx		 Reordered named imports from '@/assets'; removed explicit React import to use automatic JSX runtime; no logic or API changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • added tg footer #1144 — Also modifies Footer asset imports and related assets exports, indicating overlapping concerns in icon sourcing.
  • Sync Landing page changes #1111 — Alters exports in the same illustrations index module, suggesting related export-surface maintenance.

Suggested labels

enhancement

Suggested reviewers

  • Zishan-7
  • Hugo0
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 1edb573 and ece694d.

📒 Files selected for processing (2)
  • src/assets/illustrations/index.ts (1 hunks)
  • src/components/LandingPage/Footer.tsx (1 hunks)
✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/conflicts

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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore or @coderabbit ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@jjramirezn jjramirezn merged commit b03f8aa into peanut-wallet-dev Sep 3, 2025
8 of 9 checks passed
@coderabbitai coderabbitai bot added the enhancement New feature or request label Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jjramirezn jjramirezn jjramirezn approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@kushagrasarathe @jjramirezn @Zishan-7 @Hugo0