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

Example App - Live Update Deny #170

Merged
merged 1 commit into from
Dec 8, 2023
Merged

Conversation

alexrisch
Copy link
Contributor

@alexrisch alexrisch commented Dec 6, 2023

Resolves #160
Added request after denying the conversation to set state on the conversation item
Made wallet string selectable for easier developing
Fixed Android Button styling on home screen

Screen.Recording.2023-12-06.at.3.23.06.PM.mov

@alexrisch alexrisch requested a review from a team as a code owner December 6, 2023 22:40
Copy link
Contributor

@cameronvoell cameronvoell left a comment

Choose a reason for hiding this comment

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

The live update of denied status works great, nice fix!

Color update on Android is helpful as well.

I noticed that the selectable prop does not seem to be working for me on Android (tested on emulator and device, neither worked for me). It does make the text copy-able on iOS though, so still an improvement. Perhaps we can look at adding a copy button in some follow up PR later, since that prop seems to have some issues on Android.

Nice work!

Added request after denying the conversation to set state on the conversation item
Made wallet string selectable for easier developing
Fixed Android Button styling on home screen
@alexrisch alexrisch force-pushed the user/alexrisch/live-update-deny branch from 15e2792 to b6bb12e Compare December 7, 2023 03:49
Copy link
Contributor

@nplasterer nplasterer left a comment

Choose a reason for hiding this comment

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

LGTM! If you're looking for an easy way to get the addresses when you create a wallet they show up in the console to make it easier for copying.

@alexrisch alexrisch merged commit 061baa8 into main Dec 8, 2023
4 of 5 checks passed
@alexrisch alexrisch deleted the user/alexrisch/live-update-deny branch December 8, 2023 21:37
Copy link
Contributor

🎉 This PR is included in version 1.22.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Example app: Live-update deny/allow
3 participants