feat: add Markets page ad banner#5627
Conversation
🦋 Changeset detectedLatest commit: 7295522 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile SummaryAdds a dismissible promotional banner for the USDT Fixed-Term Vault to the Markets page header. The banner is gated behind a new per-chain user setting (
Confidence Score: 5/5Safe to merge — the change is additive, the dismissal logic is correctly wired through the header guard, and no existing flows are broken. The banner is purely additive and isolated behind a feature flag in the user's chain settings. The header early-return guard correctly handles both the show and dismissed states. apps/evm/src/containers/Layout/Header/MarketsAdBanner/index.tsx — naming inconsistencies and the inaccessible close button live here. Important Files Changed
Reviews (2): Last reviewed commit: "feat: add Markets page ad banner" | Re-trigger Greptile |
af235d2 to
1d64b67
Compare
1d64b67 to
2e95e8e
Compare
2e95e8e to
31ec3a6
Compare
Coverage Report for ./apps/evm
File Coverage
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@greptile |
31ec3a6 to
b1292a2
Compare
b1292a2 to
3123c14
Compare
3123c14 to
7295522
Compare
Jira ticket(s)
VPD-1385
Changes