Skip to content

Commit

Permalink
Merge pull request #20 from chain-notes-brussels/fixes
Browse files Browse the repository at this point in the history
finished my thoughts
  • Loading branch information
jacksmithinsulander authored Jul 13, 2024
2 parents 9edc555 + 10b87d2 commit 49407d9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ We created a metamask snap ([in a seperate repo](https://github.com/chain-notes-

### Filecoin

Since these written notes are long string, it is not an option to store them on an evm chain. We didn't want to opt into centralized hosting for this either since we wanted to build a decantralized and tamperproof system. Because of this, we took inspiration from erc721's usage of IPFS and deployed our notes, along with some more data, as JSON to IPFS, and are only storing the ipfs URI on chain as a note identification on what to retrieve for the snap. We are happy with this design and
Since these written notes are long string, it is not an option to store them on an evm chain. We didn't want to opt into centralized hosting for this either since we wanted to build a decantralized and tamperproof system. Because of this, we took inspiration from erc721's usage of IPFS and deployed our notes, along with some more data, as JSON to IPFS, and are only storing the ipfs URI on chain as a note identification on what to retrieve for the snap. We decided early on this would be the most logical way to do it, and are very happy with this design.

### Base

Expand Down

0 comments on commit 49407d9

Please sign in to comment.