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

fix(docs): update bech32 conversion docs #4436

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Thoralf-M
Copy link
Member

Description of change

Remove the part to strip away the 6 char checksum, as this is done as part of the bech32 decoding process already and the output doesn't contain a checksum anymore

Type of change

  • Documentation Fix

Remove the part to strip away the 6 char checksum, as this is done as part of the bech32 decoding process already and the output doesn't contain a checksum anymore
@Thoralf-M Thoralf-M added the dev-tools Issues related to the Developer Tools Team label Dec 11, 2024
@Thoralf-M Thoralf-M requested a review from a team as a code owner December 11, 2024 09:34
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 11, 2024
@iota-ci iota-ci added the sc-platform Issues related to the Smart Contract Platform group. label Dec 11, 2024
Copy link
Contributor

✅ Vercel Preview Deployment is ready!

View Preview

Copy link
Member

@thibault-martinez thibault-martinez left a comment

Choose a reason for hiding this comment

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

I'd like to double check something before it's merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev-tools Issues related to the Developer Tools Team documentation Improvements or additions to documentation sc-platform Issues related to the Smart Contract Platform group.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants