-
Notifications
You must be signed in to change notification settings - Fork 647
Sei chain description #8651
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
base: main
Are you sure you want to change the base?
Sei chain description #8651
Conversation
Co-authored-by: yashjoisar <yashjoisar@thirdweb.com>
|
Cursor Agent can help with this pull request. Just |
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughAdded chain metadata entry for Sei Mainnet (chain ID 1329) to the chainMetaRecord object in the utils configuration file with corresponding about description. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Warning Review ran into problems🔥 ProblemsErrors were encountered while retrieving linked issues. Errors (2)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8651 +/- ##
=======================================
Coverage 52.71% 52.71%
=======================================
Files 934 934
Lines 62945 62945
Branches 4118 4118
=======================================
Hits 33180 33180
Misses 29665 29665
Partials 100 100
🚀 New features to boost your workflow:
|
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@apps/dashboard/src/app/`(app)/(dashboard)/(chain)/utils.ts:
- Around line 288-292: The about string for the chain entry with key "1329"
contains hard-coded, time‑sensitive metrics (e.g., "4B transactions", "80M
wallets", "#1 EVM chain"); update the "about" property to use evergreen wording
or replace the static metrics with dynamic values sourced from a maintained
API/CMS; locate the object with key 1329 and either change the copy to remove
specific numbers and superlatives or wire the field to read from a maintained
data source (e.g., a CMS client or metrics fetcher) and render the fetched
values instead of hard‑coding them.
| //Sei Mainnet | ||
| 1329: { | ||
| about: | ||
| "Sei is a blockchain designed for fast, cheap financial transactions, combining the network effects of Ethereum with the performance of Solana. Sei has processed more than four billion transactions across more than 80 million wallets and has become the #1 EVM chain by number of active users. The team is backed by top investors such as Multicoin, Jump, Coinbase Ventures, and Circle Ventures – and includes talent from companies like Robinhood, Google, Coinbase, Databricks, Uber, and Goldman Sachs.", | ||
| }, |
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.
Avoid hard‑coding time‑sensitive metrics and superlatives.
The “4B transactions / 80M wallets / #1 EVM chain” claims will quickly become stale and could be misleading in UI copy. Consider making this description evergreen or sourcing these metrics from a maintained API/CMS.
✏️ Suggested evergreen wording
about:
- "Sei is a blockchain designed for fast, cheap financial transactions, combining the network effects of Ethereum with the performance of Solana. Sei has processed more than four billion transactions across more than 80 million wallets and has become the `#1` EVM chain by number of active users. The team is backed by top investors such as Multicoin, Jump, Coinbase Ventures, and Circle Ventures – and includes talent from companies like Robinhood, Google, Coinbase, Databricks, Uber, and Goldman Sachs.",
+ "Sei is a blockchain designed for fast, low-cost financial transactions, combining the network effects of Ethereum with high-performance execution. Backed by leading investors and built by a team with experience from top tech and fintech companies.",
},🤖 Prompt for AI Agents
In `@apps/dashboard/src/app/`(app)/(dashboard)/(chain)/utils.ts around lines 288 -
292, The about string for the chain entry with key "1329" contains hard-coded,
time‑sensitive metrics (e.g., "4B transactions", "80M wallets", "#1 EVM chain");
update the "about" property to use evergreen wording or replace the static
metrics with dynamic values sourced from a maintained API/CMS; locate the object
with key 1329 and either change the copy to remove specific numbers and
superlatives or wire the field to read from a maintained data source (e.g., a
CMS client or metrics fetcher) and render the fetched values instead of
hard‑coding them.
size-limit report 📦
|
Slack Thread
PR-Codex overview
This PR introduces information about the
Sei Mainnetto theutils.tsfile, providing a description of its capabilities and background, enhancing the documentation related to blockchain networks.Detailed summary
Sei Mainnetat line 1329.aboutdescription highlighting Sei's performance, transaction volume, active users, and backing by notable investors and talent.Summary by CodeRabbit
New Features
✏️ Tip: You can customize this high-level summary in your review settings.