Skip to content

Implement Cashout/Topups/Transfers#621

Open
Nodirbek75 wants to merge 69 commits into
mainfrom
feat/fygaro
Open

Implement Cashout/Topups/Transfers#621
Nodirbek75 wants to merge 69 commits into
mainfrom
feat/fygaro

Conversation

@Nodirbek75

Copy link
Copy Markdown
Contributor

No description provided.

islandbitcoin and others added 25 commits July 14, 2025 20:33
@Nodirbek75 Nodirbek75 self-assigned this Apr 22, 2026
@islandbitcoin islandbitcoin added the enhancement New feature or request label Apr 22, 2026
email: string
kycType: string
}) => {
toggleActivityIndicator(true)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also guard the modal submit path before calling bridgeInitiateKyc.

Suggested change
toggleActivityIndicator(true)
if (currentLevel === AccountLevel.Zero) {
Alert.alert("Account upgrade required", "You should upgrade your account to use this feature")
return
}
toggleActivityIndicator(true)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This screen is only accessible to Level 1+ users, so there’s no need to guard the modal for Level 0 users.

)

const checkAccountLevel = (type: "card" | "bankTransfer" | "cashout") => {
if (currentLevel === AccountLevel.Zero || currentLevel === AccountLevel.One) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Card top-up should allow Level 1 users.

Suggested change
if (currentLevel === AccountLevel.Zero || currentLevel === AccountLevel.One) {
if (currentLevel === AccountLevel.Zero) {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This logic is incorrect. It will break the JMD cashout flow and JMD bank top-up flow.
Card top-up already available for Level 1 users. What you wanted to do here?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I was getting the "level 2 required" error, because level zero and level one were not allowed

}

const checkBridgeKyc = (type: "topup" | "settle") => {
if (kycStatusData?.bridgeKycStatus === "pending") {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

International transfer should require Level 1+ before checking Bridge KYC status.

Suggested change
if (kycStatusData?.bridgeKycStatus === "pending") {
if (currentLevel === AccountLevel.Zero) {
Alert.alert("Account upgrade required", "You should upgrade your account to use this feature")
} else if (kycStatusData?.bridgeKycStatus === "pending") {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This screen is only accessible to Level 1+ users, so there’s no need to check for Level 0 users.

@linear

linear Bot commented Jun 7, 2026

Copy link
Copy Markdown

Nodirbek75 and others added 2 commits June 9, 2026 12:08
* Add International upgrade option

* fix: open bridge kyc modal from account type

* clean up and open bridge kyc modal on the account upgrade type screen

* clean up

* Hide the international/bridge upgrade button when KYC is approved and Show orange border on the international card when KYC is pending

* hide the international kyc button if user level zero

---------

Co-authored-by: Vandana <forge@getflash.io>
Co-authored-by: nodirbek75 <nodirbek7077@gmail.com>
@islandbitcoin islandbitcoin changed the title Implement Cashout V1 and Topups with bank transfer and Fygaro. Implement Cashout/Topups/Transfers Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants