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
Customizing OrbitDB Chapter added #132
Conversation
…ntroduction added. Thus the introduction content is finished.
…file names independently.
Chapters about the ACL are now an Appendix.
05_Customizing_OrbitDB/07_Implementing_a_custom_AccessController.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fantastic and I'm really excited to get it merged.
Here's the first batch of mostly grammatical issues. Have to go afk for a bit but I'll be back to it soon. Overall - review the rules for commas and when in doubt exclude them: https://www.btb.termiumplus.gc.ca/tpv2guides/guides/pep/index-eng.html?lang=eng&page=punct_2_commas_that_clutter
Co-authored-by: Mark Robert Henderson <henderson.mark@gmail.com>
Co-authored-by: Mark Robert Henderson <henderson.mark@gmail.com>
Co-authored-by: Mark Robert Henderson <henderson.mark@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last batch of comments! Looking better and better.
Also, why were the dist
files deleted?
|
||
It is setup when you create a database | ||
and stored in the Manifest of the database. | ||
(That thing, that the mulithash in the middle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not really but you can perhaps placeholder the hash
|
||
But this also means, you cannot modify your access | ||
controller and it's options after you created | ||
the database. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gotcha, maybe specify you're talking about the AC type, or method
Co-authored-by: Mark Robert Henderson <henderson.mark@gmail.com>
To make clear, that you cannot change the AC type after the creation of a db. Also used an example address.
Co-authored-by: Mark Robert Henderson <henderson.mark@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last batch of comments! Looking better and better.
Also, why were the
dist
files deleted?
If we use CD Actions, we shouldn't need them. But we could put it back.
these trees from the oplog or `ipfs-log`. | ||
|
||
In the index, we'll see the oplog as an | ||
Array of operations, each containing these fields: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to use Array
because I thought the oplog
passed to the Index
is of type Array
.
This isn't the case?
by choosing how we want to represent the | ||
comments in the Index. | ||
|
||
By choosing a data structure and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried to change this a bit.
No by using a numbered list, because that would have been very repetitive.
Co-authored-by: Mark Robert Henderson <henderson.mark@gmail.com>
I refer to #131 for further details on this PR.