Skip to content
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.

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

SEP-6 new flow #353

Merged
merged 5 commits into from
Jul 3, 2024
Merged

SEP-6 new flow #353

merged 5 commits into from
Jul 3, 2024

Conversation

quietbits
Copy link
Contributor

No description provided.

@stellar-jenkins
Copy link

@quietbits quietbits requested review from JakeUrban and philipliu June 27, 2024 19:47
Copy link

@philipliu philipliu left a comment

Choose a reason for hiding this comment

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

LGTM other than the deposit instructions modal.

@JakeUrban
Copy link
Contributor

Are we going to add the SEP-38 functionality in this PR or a subsequent one?

@JakeUrban
Copy link
Contributor

@quietbits and I talked offline and decided to merge this PR separately from adding quotes, we'll do that next.

* SEP-6 deposit quotes in progress

* Show only selected asset prices

* Fix quotes post assets

* SEP-38 flow updates

* Quotes deposit works

* Quotes withdrawal works
@stellar-jenkins
Copy link

@quietbits quietbits merged commit a236624 into master Jul 3, 2024
2 checks passed
@quietbits quietbits deleted the sep6-flow-update branch July 3, 2024 17:11
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.

4 participants