-
Notifications
You must be signed in to change notification settings - Fork 51
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
Remove pub, priv and pub(set) documentation #146
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Some of the updated docs reflect contracts that are currently deployed and would not reflect the versions shown in this PR, like FlowToken
, FlowIDTableStaking
, NFTStorefront
, StakingCollection
, FlowEpoch
, and the token standards.
In your description, you said you don't want to update the docs for deployed contracts until the contracts have been updated, so we should revert all of those until the contracts have been updated, right?
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.
All the tutorials have separate saved playground projects which will also need to be updated to reflect these changes
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'll create an issue for it and add it to our backlog. Not sure exactly when we'll get to it, but it should hopefully be soon
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.
cc @SupunS @joshuahannan can you take another look at this? |
With onflow/cadence#2540 merged, these access modifiers are going away in Stable Cadence.
Note that
docs/concepts/staking/05-epoch-scripts-events.md
anddocs/tooling/unity-sdk/samples/nft-browser.md
still containpub(set)
access modifiers, but given that these reflect real contracts, I did not want to update them unilaterally. Once these contracts are updated, the docs should change to match them.