Closed thulieblack closed 6 days ago
[!CAUTION]
Review failed
The pull request is closed.
The changes in this pull request involve an update to the cfaText
property within the banners
array located in the components/campaigns/banners.ts
file. The text has been updated from "Get Your Tickets" to "Get Your Free Ticket". The logic of the shouldShowBanner
function remains unchanged, which continues to determine the visibility of the banner based on the current date relative to the cfpDeadline
.
File | Change Summary |
---|---|
components/campaigns/banners.ts | Updated cfaText from "Get Your Tickets" to "Get Your Free Ticket". Logic of shouldShowBanner remains unchanged. |
cfaText
property in the banners
array, which is directly related to the change made in the main PR where the cfaText
was updated from "Get Your Tickets" to "Get Your Free Ticket".banners
array and includes changes to the shouldShowBanner
function, which is relevant as the main PR's changes occur in the same file and involve the cfaText
property.🐇 In the meadow, banners sway,
"Get Your Free Ticket!" they say.
A change so bright, a message clear,
Hop along, the conference is near!
With joy we bounce, in the sunny light,
Join the fun, it feels just right! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Name | Link |
---|---|
Latest commit | 09f5b8bcb25193e818e4ad4ceb89b3903478676a |
Latest deploy log | https://app.netlify.com/sites/asyncapi-website/deploys/673c9891c913850008835e0c |
Deploy Preview | https://deploy-preview-3412--asyncapi-website.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 67.77%. Comparing base (
6bf75fb
) to head (09f5b8b
). Report is 1 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
🚨 Try these New Features:
⚡️ Lighthouse report for the changes in this PR: | Category | Score |
---|---|---|
🔴 Performance | 32 | |
🟢 Accessibility | 98 | |
🟢 Best practices | 92 | |
🟢 SEO | 100 | |
🔴 PWA | 33 |
Lighthouse ran on https://deploy-preview-3412--asyncapi-website.netlify.app/
/ptal
@derberg @magicmatatjahu @devilkiller-ag @akshatnema @sambhavgupta0705 @anshgoyalevil @Mayaleeeee Please take a look at this PR. Thanks! :wave:
/rtm
Change the ticket button description, to specify that it's free
Summary by CodeRabbit
New Features
Bug Fixes