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

feat(sdk): implement official CBOR based metadata #96

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

Conversation

iamcrazycoder
Copy link
Contributor

What this PR does / why we need it:

This PR implements the the official CBOR-based metadata. Once merged, Ordit-SDK will be bidding goodbye to OIP-1.

@iamcrazycoder iamcrazycoder self-assigned this Nov 8, 2023
@iamcrazycoder iamcrazycoder changed the title feat(sdk): feat: implement official CBOR based metadata feat(sdk): implement official CBOR based metadata Nov 8, 2023
@iamcrazycoder iamcrazycoder marked this pull request as ready for review November 8, 2023 12:39
@@ -30,6 +30,7 @@
"bitcoinjs-lib": "6.1.3",
"bitcoinjs-message": "2.2.0",
"buffer-reverse": "^1.0.1",
"cbor-js": "^0.1.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we replace it with this? https://www.npmjs.com/package/cbor

chor-js was last released 8 years ago.

@yubing744
Copy link

@iamcrazycoder Hello, I think this feature is very good. Can I merge it into the main branch and publish a version to the npm repository as soon as possible?

@thedoublejay
Copy link
Contributor

@yubing744 we cannot merge it yet as it will break some of our metadata. So we put it on hold, not our priority atm. But please feel free to fork it if needed. thank you!

@yubing744
Copy link

@yubing744 we cannot merge it yet as it will break some of our metadata. So we put it on hold, not our priority atm. But please feel free to fork it if needed. thank you!

OK, I forked this project and wrote another PR to temporarily meet the needs of our bitseed project.

@thedoublejay
Copy link
Contributor

Thank you @yubing744! We will reach out once we need to work on this. It's on our timeline

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.

3 participants