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

docs: New Supported networks page #610

Merged
merged 29 commits into from
Oct 17, 2024
Merged

docs: New Supported networks page #610

merged 29 commits into from
Oct 17, 2024

Conversation

louis-md
Copy link
Collaborator

@louis-md louis-md commented Oct 2, 2024

Unify all "Supported networks" pages across the site into a better, discoverable design.

Copy link

github-actions bot commented Oct 2, 2024

Branch preview

✅ Deployed successfully in branch deployment:

https://supported_networks--docs.review.5afe.dev

Copy link

github-actions bot commented Oct 2, 2024

Overall readability score: 31.94 (🔴 -6.82)

File Readability
smart-account-bug-bounty.md 53.46 (🟢 +0)
smart-account-supported-networks.mdx 0 (-)
arbitrum.mdx 0 (-)
aurora.mdx 0 (-)
avalanche.mdx 0 (-)
base.mdx 0 (-)
blast.mdx 0 (-)
bsc.mdx 0 (🟢 +0)
celo.mdx 0 (-)
chiado.mdx 0 (-)
gnosis.mdx 0 (-)
linea.mdx 0 (-)
mainnet.mdx 0 (-)
mantle.mdx 0 (-)
optimism.mdx 0 (-)
polygon.mdx 0 (-)
scroll.mdx 0 (-)
sepolia.mdx 0 (-)
worldchain.mdx 0 (-)
xlayer.mdx 0 (-)
zkevm.mdx 0 (-)
zksync.mdx 0 (-)
View detailed metrics

🟢 - Shows an increase in readability
🔴 - Shows a decrease in readability

File Readability FRE GF ARI CLI DCRS
smart-account-bug-bounty.md 53.46 39.53 10.78 12.4 13.45 7.56
  🟢 +0 🟢 +0 🟢 +0 🟢 +0 🟢 +0 🟢 +0
smart-account-supported-networks.mdx 0 0 19 22 19 11
  - - - - - -
arbitrum.mdx 0 0 19 22 19 11
  - - - - - -
aurora.mdx 0 0 19 22 19 11
  - - - - - -
avalanche.mdx 0 0 19 22 19 11
  - - - - - -
base.mdx 0 0 19 22 19 11
  - - - - - -
blast.mdx 0 0 19 22 19 11
  - - - - - -
bsc.mdx 0 0 19 22 19 11
  🟢 +0 🟢 +0 🟢 +0 🟢 +0 🟢 +0 🟢 +0
celo.mdx 0 0 19 22 19 11
  - - - - - -
chiado.mdx 0 0 19 22 19 11
  - - - - - -
gnosis.mdx 0 0 19 22 19 11
  - - - - - -
linea.mdx 0 0 19 22 19 11
  - - - - - -
mainnet.mdx 0 0 19 22 19 11
  - - - - - -
mantle.mdx 0 0 19 22 19 11
  - - - - - -
optimism.mdx 0 0 19 22 19 11
  - - - - - -
polygon.mdx 0 0 19 22 19 11
  - - - - - -
scroll.mdx 0 0 19 22 19 11
  - - - - - -
sepolia.mdx 0 0 19 22 19 11
  - - - - - -
worldchain.mdx 0 0 19 22 19 11
  - - - - - -
xlayer.mdx 0 0 19 22 19 11
  - - - - - -
zkevm.mdx 0 0 19 22 19 11
  - - - - - -
zksync.mdx 0 0 19 22 19 11
  - - - - - -

Averages:

  Readability FRE GF ARI CLI DCRS
Average 31.94 27.65 13.57 17.66 15.64 8.82
  🔴 -6.82 🔴 -6.51 🔴 -0.99 🔴 -1.09 🔴 -0.81 🔴 -0.41
View metric targets
Metric Range Ideal score
Flesch Reading Ease 100 (very easy read) to 0 (extremely difficult read) 60
Gunning Fog 6 (very easy read) to 17 (extremely difficult read) 8 or less
Auto. Read. Index 6 (very easy read) to 14 (extremely difficult read) 8 or less
Coleman Liau Index 6 (very easy read) to 17 (extremely difficult read) 8 or less
Dale-Chall Readability 4.9 (very easy read) to 9.9 (extremely difficult read) 6.9 or less

Copy link
Collaborator

@tanay1337 tanay1337 left a comment

Choose a reason for hiding this comment

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

@louis-md can you please update all the "Supported Network" links (from the Features section and the API section)? They give a 404 right now.

Can you also please pull in the latest changes from main, so that we can run the Vale workflow as well?

@louis-md
Copy link
Collaborator Author

@tanay1337 I just fixed some broken links, however the two that you linked (from link checker) will still error as the link checker cannot handle links with a query string in the relative path (if you try the link manually it works though).

Copy link
Collaborator

@tanay1337 tanay1337 left a comment

Choose a reason for hiding this comment

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

  • The links to the page (from 4337, API, etc.) don't have the filter applied. As an example, see this link.
  • The lazy loading does not work properly - it will unveil three pages and then stop. There's no way to view the whole list.

@tanay1337 tanay1337 merged commit 904a833 into main Oct 17, 2024
3 of 4 checks passed
@tanay1337 tanay1337 deleted the supported-networks branch October 17, 2024 07:18
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.

2 participants