Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ft/search params and links #4

Open
wants to merge 8 commits into
base: stage
Choose a base branch
from
Open

Conversation

hirwaaldo1
Copy link
Owner

This PR covers the following content (from scrimba)

  1. Search Params Intro
  2. useSearchParams
  3. Challenge: Set up search params in VanLife
  4. Filter the array w/ the search param
  5. Challenge: Filter the vans in VanLife
  6. using Links to add search params
  7. Challenge: Filter the vans with Links
  8. Using the search params setter function
  9. Challenge: Filter the vans with a setter function
  10. Caveats to setting params
  11. Merging search params with Links
  12. Merging search params with the setSearchParams function
  13. Challenge: Conditional rendering practice
  14. Fix remaining absolute paths
  15. Back to all vans
  16. Link state
  17. useLocation
  18. Challenge: conditionally render the back button text
  19. 19.404 Page
  20. "Happy Path" vs. "Sad Path"
  21. Quick update to our fetching code
  22. Coding the Sad Path - Loading state
  23. Coding the Sad Path - Error handling

@netlify
Copy link

netlify bot commented Apr 6, 2023

Deploy Preview for a-van ready!

Name Link
🔨 Latest commit 38fd616
🔍 Latest deploy log https://app.netlify.com/sites/a-van/deploys/64401f4034ee280008eff0bc
😎 Deploy Preview https://deploy-preview-4--a-van.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@hirwaaldo1 hirwaaldo1 added the Ready for code review Ready for any one who want to review label Apr 6, 2023
src/pages/Host/Vans.jsx Outdated Show resolved Hide resolved
src/pages/Vans/Detail.jsx Outdated Show resolved Hide resolved
src/pages/Vans/Detail.jsx Outdated Show resolved Hide resolved
src/pages/Host/Vans.jsx Outdated Show resolved Hide resolved
src/pages/Vans/index.jsx Outdated Show resolved Hide resolved
@hirwaaldo1 hirwaaldo1 added Done Done and removed Ready for code review Ready for any one who want to review labels Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Done Done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants