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!: Make link.toJSON return a DAG-JSON link #230

Merged
merged 1 commit into from
Jan 2, 2023
Merged

Conversation

Gozala
Copy link
Contributor

@Gozala Gozala commented Dec 13, 2022

Changes toJSON implementation of the CID so it returns Link in DAG-JSON format as opposed to object with Uint8Array which gets munged with JSON.stringify

Fixes #228

Copy link
Contributor

@mikeal mikeal left a comment

Choose a reason for hiding this comment

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

We should bump the major version of multiformats for this release. Hopefully the automating picks it up w/ “feat!”

@RangerMauve
Copy link
Collaborator

Mind if we push this to January? Rod is off for the next few weeks and it'd be good to get his input.

Copy link
Member

@achingbrain achingbrain left a comment

Choose a reason for hiding this comment

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

LGTM

@rvagg rvagg merged commit 9e087d6 into master Jan 2, 2023
@rvagg rvagg deleted the feat/link-to-json branch January 2, 2023 06:10
achingbrain added a commit to achingbrain/js-libp2p-noise that referenced this pull request Jan 6, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230)
so update all deps using multiformats to the latest version.

The code changed is not used by this module so there's not a lot of
changes here, but this should still go out as a major because the
interface versions have changed.
achingbrain added a commit to achingbrain/gossipsub-js that referenced this pull request Jan 6, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using
multiformats to the latest version.

The code changed is not used by this module so there's not a lot of changes here, but this should still go out as a major because the interface
versions have changed.
achingbrain added a commit to libp2p/js-libp2p that referenced this pull request Jan 6, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.
wemeetagain pushed a commit to ChainSafe/js-libp2p-noise that referenced this pull request Jan 6, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230)
so update all deps using multiformats to the latest version.

The code changed is not used by this module so there's not a lot of
changes here, but this should still go out as a major because the
interface versions have changed.
wemeetagain pushed a commit to ChainSafe/js-libp2p-gossipsub that referenced this pull request Jan 6, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using
multiformats to the latest version.

The code changed is not used by this module so there's not a lot of changes here, but this should still go out as a major because the interface
versions have changed.
achingbrain added a commit to libp2p/js-libp2p-daemon that referenced this pull request Jan 7, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.

Co-authored-by: achingbrain <[email protected]>
achingbrain pushed a commit to libp2p/interop that referenced this pull request Jan 7, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.
achingbrain added a commit to ipfs/js-datastore-pubsub that referenced this pull request Jan 7, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using
multiformats to the latest version.
achingbrain added a commit to libp2p/js-libp2p that referenced this pull request Jan 7, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) to the return type of the `.toJSON` method of the `CID` class so update all deps using multiformats to the latest version.
achingbrain added a commit to ipfs/js-datastore-pubsub that referenced this pull request Jan 7, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.
achingbrain added a commit to ipfs/js-ipfs-bitswap that referenced this pull request Jan 7, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to
the latest version.
achingbrain added a commit to ipfs/js-ipfs-bitswap that referenced this pull request Jan 7, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.
achingbrain added a commit to ipfs/js-ipfs that referenced this pull request Jan 7, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.
achingbrain added a commit to achingbrain/js-dag-jose that referenced this pull request Jan 9, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.

BREAKING CHANGE: the CID class exposed by the types of this module now comes from `[email protected]`
achingbrain added a commit to achingbrain/js-dag-jose that referenced this pull request Jan 9, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.

BREAKING CHANGE: the CID class exposed by the types of this module now comes from `[email protected]`
achingbrain added a commit to ipfs/js-ipfs that referenced this pull request Jan 9, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.
ukstv pushed a commit to ceramicnetwork/js-dag-jose that referenced this pull request Jan 11, 2023
`[email protected]` shipped with a [breaking change](multiformats/js-multiformats#230) so update all deps using multiformats to the latest version.

BREAKING CHANGE: the CID class exposed by the types of this module now comes from `[email protected]`
achingbrain added a commit to ipfs/js-kubo-rpc-client that referenced this pull request Jan 13, 2023
[email protected] shipped with a [breaking change to the CID class](multiformats/js-multiformats#230).

This change updates all dependencies to use [email protected].

BREAKING CHANGE: instances of the CID class returned by this module are now from [email protected]

Co-authored-by: achingbrain <[email protected]>
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.

Improve toJSON of the CID
5 participants