Skip to content
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

Add L1 to L2 upgrade decision tree #1465

Merged
merged 3 commits into from
Sep 20, 2024
Merged

Add L1 to L2 upgrade decision tree #1465

merged 3 commits into from
Sep 20, 2024

Conversation

lvpeschke
Copy link
Contributor

@lvpeschke lvpeschke commented Sep 18, 2024

For anyone unsure about what they should be doing for the migration, a landing page to help understand next actions.

closes https://github.com/celo-org/celo-blockchain-planning/issues/572

Add content from the [decision tree doc](https://docs.google.com/document/d/1_IGhGeVRSQYLZT9qfm66cagrM8Tgh1qZvyCEp9zSnes/edit) to the docs with HTML details to toggle relevant info while keeping it searchable.
Copy link

netlify bot commented Sep 18, 2024

Deploy Preview for celo-docs ready!

Name Link
🔨 Latest commit d9c32a5
🔍 Latest deploy log https://app.netlify.com/sites/celo-docs/deploys/66eb01619a3c1f0008016491
😎 Deploy Preview https://deploy-preview-1465--celo-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 35 (no change from production)
Accessibility: 92 (no change from production)
Best Practices: 75 (no change from production)
SEO: 82 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@lvpeschke lvpeschke closed this Sep 19, 2024
@lvpeschke lvpeschke reopened this Sep 19, 2024
@lvpeschke lvpeschke marked this pull request as ready for review September 19, 2024 22:50
@lvpeschke lvpeschke requested a review from a team as a code owner September 19, 2024 22:50
@lvpeschke lvpeschke enabled auto-merge (squash) September 19, 2024 22:51
<details>
<summary>I use ContractKit or Fee Abstraction (ERC-20 gas tokens)</summary>

Recommended versions for SDK and libraries:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it's required, I would change it to "required"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not, strictly speaking. The required versions are much lower, but @aaronmboyd made a good point in not encouraging people to use old versions at all and always point them to the latest.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That might have been @aaronmgdr hehe

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True - Aarons everywhere 😆

Reach out to us!

- Technical question? [celo-org discussions on GitHub](https://github.com/orgs/celo-org/discussions/categories/cel2)
- Any type of question? TODO Discord channel
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add. Please let me know if I should communicate that

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also add link to Discord

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The channel didn't exist at the time of writing 🙃 can add as soon as I can link to it

@lvpeschke lvpeschke merged commit 1fdcd06 into main Sep 20, 2024
7 checks passed
@lvpeschke lvpeschke deleted the lvpeschke/decision-tree branch September 20, 2024 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants