Closed dipanshurdev closed 9 months ago
@amanr-dev is attempting to deploy a commit to the EddieHub Team on Vercel.
A member of the Team first needs to authorize it.
The recent update enhances the user experience by incorporating repository information into alert messages and URL query parameters. When users perform actions that trigger alerts, the repository name is now displayed, and the success message URL includes the repository owner and name, providing a more informative and personalized interaction.
File(s) | Change Summary |
---|---|
src/app/page.js |
Added a repo variable to capture the "repo" parameter from the URL query string. Modified the Toast component to include the repo prop when alert is truthy. Extended the timeout duration for the router.push call from 4000 to 5000 milliseconds when the alert is truthy. |
src/components/Toast.js |
Modified the Toast function to wrap the message in a span with the classes "font-bold", and to split the message into two parts, with the first 18 characters in one span and the rest in another span with the class "text-white". Changed the class of the span wrapping the message from "font-semibold" to "font-bold". |
src/app/rate/Form.js |
The redirect function call within the Form component has been modified to include additional query parameters for the success message, incorporating the repository owner and name into the redirect URL. |
Objective | Addressed | Explanation |
---|---|---|
Show Repo name in alert (#93) | ✅ | |
Include Owner and Repo's name in search query | ✅ |
🐇 Here's to the code that we've spun,
With each push, we've all but won.
Alerts now with a repo's name,
A rater's journey, never the same. 🌟
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?
Hi @eddiejaoude , please review and test the PR👍
Thank you for the contribution.
By adding another parameter to the toast component, we cannot use it for anything else. Please update the return string to contain the whole message, so the toast component is more generic and reusable
Removed the "repo" param.
Little bit javascript for changing the text color and it's tested.
Fixes Issue
Fixed #93
Changes proposed
Update query params in URL(Form), Tost, and Page Components.
Check List (Check all the applicable boxes)
Screenshots
Note to reviewers
Added some styling
Summary by CodeRabbit
New Features
Improvements