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

v0.11 fixes #55

Closed
wants to merge 1 commit into from
Closed

v0.11 fixes #55

wants to merge 1 commit into from

Conversation

zoedberg
Copy link
Contributor

This PR is part of a v0.11 fixes epic, see RGB-WG/rgb-tests#30 for an overview.

Copy link
Member

@dr-orlovsky dr-orlovsky left a comment

Choose a reason for hiding this comment

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

Concept NACK: we do not need non-standard Baid64 encoding, which is not used in bitcoin, to be a part of this library. Also, non-standard encoding of addresses doesn't belong here.

This library must know and assume nothing about RGB - it was designed in this way and must remain so.

@zoedberg
Copy link
Contributor Author

Sorry, I thought that by removing the close method this became a structure for standard bitcoin addresses. I will move it back to where it was before.

@zoedberg
Copy link
Contributor Author

This should be not needed anymore, closing

@zoedberg zoedberg closed this Jan 16, 2025
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.

2 participants