search

blockchain-lab-um / masca

Snap for managing VCs and DIDs in MetaMask
https://masca.io
Apache License 2.0
54 stars 16 forks source link

feat: pin ceramic streams on merge action #536

Closed pseudobun closed 10 months ago

pseudobun commented 10 months ago

Currently this will have to do since we are still using barebones Ceramic Netwok. When we migrate to ComposeDB, this will be handled more conveniently; indexed Models in ComposeDB automatically stay pinned.

changeset-bot[bot] commented 10 months ago

⚠️ No Changeset found

Latest commit: f50f77564804f2cc047d6ebaa780d4d93fe571ac

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

pseudobun commented 10 months ago

The node pins the stream by default when it's added. Streams should be pinned explicitly only when they are loaded from a different (foreign) node. This PR is therefore not needed.