-
Notifications
You must be signed in to change notification settings - Fork 152
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
feat: View saved addresses in BNMO checkout #6920
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥇 🚀 This looks great 💯 A couple of small comments and suggestions. I will pull this PR to do some additional QA before merging it.
f88547f
to
d071a2a
Compare
dd39735
to
782c51d
Compare
I did some more QA and it works great! 🎉 🥇 I found a small problem with populating the country name when 2 saved addresses have different countries but definitely doesn't need to be fixed in this PR (I just wrote it here as a note for myself to check it when working on the Add/Edit address part) |
d071a2a
to
e1e1c4b
Compare
Merged. Thanks again 🙏🏽 |
Updated the schema in Force to get access to the
addressConnection
. Edited the UI to display the list of addresses and populate them in the component's state when selected.https://artsyproduct.atlassian.net/browse/PURCHASE-2300