Facuzeta/adding banner with new sites #10
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds an informational banner to notify users that orderflow.art is no longer being updated and directs them to alternative sites for current data. The changes include upgrading the Next.js dependency to allow patch updates and implementing a styled banner component with links to Allium and Barter platforms.
Key Changes
- Updated Next.js version constraint from exact to caret (allowing patch updates)
- Added informational banner displaying the data coverage period and deprecation notice
- Integrated links to two alternative platforms (Allium and Barter) with supported chain information
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Changed Next.js version from exact "14.0.4" to "^14.0.4" to allow patch updates |
| components/sankey/Render.tsx | Added deprecation banner with data range, alternative site links, and imported getDateString helper |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| error={sankeyError} | ||
| /> | ||
| </div> | ||
| <div className="border border-dune-300 bg-dune-200 px-6 py-6 rounded-md text-center space-y-6"> |
There was a problem hiding this comment.
The added banner is missing a top margin or spacing, which could cause it to be immediately adjacent to the element above. Consider adding a top margin (e.g., 'mt-4' or 'mt-6') to maintain consistent spacing with the surrounding layout.
| <div className="border border-dune-300 bg-dune-200 px-6 py-6 rounded-md text-center space-y-6"> | |
| <div className="border border-dune-300 bg-dune-200 px-6 py-6 rounded-md text-center space-y-6 mt-6"> |
2 participants
Adding banner with new sites