-
Couldn't load subscription status.
- Fork 20
refactor: flow status page #386
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
Conversation
5185958 to
a29b7a8
Compare
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.
👍
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.
saw that you just merged it, but i was actually reviewing it now and it looks good👍
|
Sorry for merging it without your review. I will not do that for other PRs. |
np np, you don't have to, i was doing it because you requested a review |
* update custom deployments page (#382) * update custom deployments page * remove just as requirement from custom deployments * andrei's feedback * add step 4 for comptroller --------- Co-authored-by: Andrei Vlad Birgaoanu <andreivladbrg@gmail.com> * style: add overrides for md and mdx * refactor: flow status page (#386) * chore: use snapshot time instead of start time in Stream characteristics table * Bump sdk and update governance page (#372) * bump sdk * add latest contract deployments * bump sdk * feat: comptrollers table refactor: hardcode values instead of making rpc calls refactor: update governance page * refactor: update supported chains chore: add PR NOTEs * chore: nitpick * bump sablier * chore: add todo about reference links chore: bump deps * chore: fix deps * Feedback on PR #372 (#388) * feat: add getLatestLockupChainIds helper function refactor: getChainCount function to compute the count on its own refactor: query comptroller address from sdk chore: update words on the governance page refactor: create snippet for deployment note * capitalize comptroller remove redundant check from ComptrollersTable --------- Co-authored-by: smol-ninja <shubhamy2015@gmail.com> * feat: add contract references (#383) * docs: add contract references and update autogen script using Claude * fix: add evm-utils to gitmodules script: update autogen to deal with malformed code block endings with asterisks * feat: add concepts section for variable claim airdrop (#387) * feat: add concepts section for variable claim airdrop * update diagrams and add VCA campaign update access control and add claimTo and ClaimViaSig * andrei's suggestion * docs: add use cases for VCA airdrop docs: use claimTo in VCA diagram docs: add compatibility column to airdrops access control * refactor: update lockup examples with latest version (#391) * refactor: update lockup examples with latest version refactor: update example GH url links refactor: remove staking page * change type import * fix conflicts --------- Co-authored-by: smol-ninja <shubhamy2015@gmail.com> * revert from warn to throw * refactor: update airdrops examples (#392) * refactor: update airdrops examples * refactor: hide campaign verifiction page * chore: wordings * fix formatting --------- Co-authored-by: smol-ninja <shubhamy2015@gmail.com> * fix formatting * fix: broken links * refactor: update flow examples * fix: broke links * remove Broker from flow example * update lockfile * fix: vercel issue --------- Co-authored-by: Shubham Yadav <shubhamy2015@gmail.com>
I removed
Lifecyclesection from Flow overview as I feel its a repetition of Flow diagrams from the Status page.