Add Experience Details Page #30
Conversation
NitBravoA92
added
documentation
There was a problem hiding this comment.
Status: Approved 🟢
Hi @NitBravoA92
Great work on implementing the new feature, this is complete so it's time to merge it 💯 🥳.
Highlights 🥇
- Linters running
- The app works as expected
- Good and clean design
Optional
All the changes with optional tags are not crucial enough to prevent you from getting the approval. Still, I would highly recommend that you consider these changes.
Happy coding! 👏 👏 👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Remember to tag me in your question so I can receive the notification.
| <div className="w-100"> | ||
| <div className="experience-details w-100"> | ||
| <div className="experience-image w-100"> | ||
| <img src={selectedExperience?.image} alt={selectedExperience?.name} className="fluid-img" /> |
There was a problem hiding this comment.
[Optional] I recommended you decrease the size of the image on the desktop version, to 768px and consider blocking the size at 600px.
There was a problem hiding this comment.
@AndreaM2429 Thank you very much for taking the time to review my PR and suggesting these valuable changes. They have been added.
🚩Experience Details page
Hi @AndreaM2429
I'm sending this pull request for you to review my code. Thanks in advance for your feedback!
Milestones 🎯:
📌 Generate the
ExperienceDetails.js, and added to thesrc/Componentsfolder📌 Generate the
ExperienceIncluded.js, and added to thesrc/Componentsfolder📌 Create the helper method:
getRandomDiscount()to generate a random discount📌 Get the selected
experiencedata from the store using theexperienceIDand display it on theExperience Detailspage📌 Update the
App.jsfile to include the new CSS styles and theExperienceDetailscomponent📌 Update the main title of the App in the
index.htmlfile📌 Create the CSS styles to match the original design as closely as possible.
📌 Update the code in the files:
RoutesWrapper.js,usersSlice.js,MainPage.jsxandReservationFormto fix some bugs in the process of loading the pages⭐To the code reviewer 👨💻
Please, I kindly ask you to review the entire code and verify that all requirements have been met correctly.
if there is any issue 🦯 in this PR, please do list 📃 them in a descriptive 💡 manner and give your best suggestions 🎁 if needed.