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

[BRO-13] Payload decoding in the browser wallet of the CIS3 standard #83

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Ivan-Mahda
Copy link

Update examples with new signCIS3Message

Purpose

With sign CIS3 message, we can explicitly sign new message standard and decode payload. But in this case we should send all additional data as message parameters, because end result message will be created at browser wallet side.

Checklist

  • My code follows the style of this project.
  • The code compiles without warnings.
  • I have performed a self-review of the changes.
  • I have documented my code, in particular the intent of the
    hard-to-understand areas.
  • (If necessary) I have updated the CHANGELOG.

By submitting the contribution I accept the terms and conditions of the
Contributor License Agreement v1.0

@Ivan-Mahda Ivan-Mahda requested a review from DOBEN April 23, 2024 16:27
Update CHANGELOG.md, README.md, dApp version
Updated function names
Removed unused constants
@@ -1,5 +1,9 @@
## Unreleased changes

## 2.0.3

- Change the dApp to use new signCIS3Message from BrowserWallet
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Change the dApp to use new signCIS3Message from BrowserWallet
- Change the dApp to use new `signCIS3Message` function from the BrowserWallet.

@@ -1,5 +1,9 @@
## Unreleased changes

## 1.0.4

- Change the dApp to use new signCIS3Message from BrowserWallet
Copy link
Member

Choose a reason for hiding this comment

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

  • Change the dApp to use new signCIS3Message function from the BrowserWallet.

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.

None yet

2 participants