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

Cards updates: WIP #465

Merged
merged 22 commits into from
Apr 5, 2024
Merged

Cards updates: WIP #465

merged 22 commits into from
Apr 5, 2024

Conversation

mapachurro
Copy link
Collaborator

@mapachurro mapachurro commented Mar 6, 2024

Adding cards index files across the site.

  • Might be I believe it is complete in terms of every directory having a card
  • Cards are not I believe cards are all properly named
  • Moved Run a Node up to top level, as per Declan's request

To-Do, and out of scope for this initial version:

Copy link

vercel bot commented Mar 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
doc-zk-evm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 5, 2024 9:34am

@mapachurro mapachurro marked this pull request as ready for review March 20, 2024 12:09
@bgravenorst
Copy link
Collaborator

Hey @mapachurro, I may have misunderstood, but my assumption was that Declan wanted the "Run a node" heading in the horizontal heading alongside "Linea basic", "Developers", "Linea architecture".

docs/use-mainnet/bridges-of-linea/index.mdx Outdated Show resolved Hide resolved
sidebars.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@jlwllmr jlwllmr left a comment

Choose a reason for hiding this comment

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

One minor comment about the necessity for the top-level sidebar category, but otherwise fine

sidebars.js Show resolved Hide resolved
@jlwllmr jlwllmr merged commit 7d6f64d into main Apr 5, 2024
7 checks passed
@jlwllmr jlwllmr deleted the cards-updates-2 branch April 5, 2024 09:35
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.

3 participants