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

Exchanging messages with Gnokey Mobile #133

Merged
merged 23 commits into from
Oct 22, 2024
Merged

Conversation

iuricmp
Copy link
Contributor

@iuricmp iuricmp commented Oct 3, 2024

  • Prepares dSocial to send and to receive Linking Parameters from the operational system.
  • Updates "@gnolang/gnonative" to "3.0.0"

How to test:

  1. Open dSocial and Gnokey mobile
  2. Create accounts in both of them
  3. Try to Post a message using dSocial
  4. After pressing the Post button, dSocial will open Gnokey Mobile and expect you select a key to proceed.
  5. After selecting a key in Gnokey, it will navigate back do dSocial.
  6. dSocial will receive the key address you selected and will use it to create a unsigned transaction.
  7. The unsigned transaction will be sent to Gnokey to be signed.
  8. Gnokey requires you to select a key to sign the transaction. Use the same key you selected before.
  9. The signed transaction is sent back to dSocial.
  10. dSocial will broadcast the transaction to be inserted in the blockchain.

@iuricmp iuricmp requested review from D4ryl00 and jefft0 October 4, 2024 07:55
@iuricmp iuricmp marked this pull request as ready for review October 4, 2024 07:55
@iuricmp iuricmp changed the title Inter-app-communication Exchanging messages with Gnokey Mobile Oct 4, 2024
@jefft0
Copy link
Contributor

jefft0 commented Oct 16, 2024

I pushed a commit to fix the Following bug. user.address.toString() should be user.bech32, on this line:
https://github.com/gnolang/dsocial/blob/inter-app-communication/mobile/components/view/account/index.tsx#L64

2daa7f9

@iuricmp iuricmp force-pushed the inter-app-communication branch from a73bd94 to 2864453 Compare October 21, 2024 12:29
Copy link
Contributor

@jefft0 jefft0 left a comment

Choose a reason for hiding this comment

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

Tested on Android and iOS simulators. See my suggested fix for "Avatar".

@iuricmp
Copy link
Contributor Author

iuricmp commented Oct 22, 2024

That’s it.

@iuricmp iuricmp merged commit f00e4a0 into main Oct 22, 2024
1 check passed
Copy link

🎉 This PR is included in version 1.11.0 🎉

The release is available on GitHub release

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.

2 participants