palaviajeros / palaviajeros-ui

https://palaviajeros.netlify.app/
0 stars 0 forks source link

#117: update best seller cards to accept a function to filter travelpackages, #123 Consumed this new method to create featured packages slider and hot deals slider #122

Closed jeandreidc closed 2 months ago

jeandreidc commented 2 months ago

Resolves #117

Done

  1. Refactored package actions to include a filter
  2. Updated best-seller page to use this method
  3. Added new tags field for TravelPackage to contain tags such as: best-seller, sale 3.1. We can use these tags to determine if a package is on sale, featured or best-seller 3.2. Need to update the parser to include this new field. Created this ticket to track it
  4. Converted actions/travelPackages to only be executed in the server
  5. Refactored methods of travelPackages actions to be async

Improvements

Resolves #123

Done

  1. After implementing findPackagesPerCountry() it opened up a flexible way of fetching packages.
  2. Also introduced tags concept that we could use to filter the packages in whichever way we like
  3. Created a generic PackageSlider and PackageSliderCard that accepts parameters and filters

Screenshots

image

github-actions[bot] commented 2 months ago

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

netlify[bot] commented 2 months ago

Deploy Preview for palaviajeros ready!

Name Link
Latest commit 9d12516b3856b9468303aac2efd5348ee7f83266
Latest deploy log https://app.netlify.com/sites/palaviajeros/deploys/66b7cf5371d4a600089ed74a
Deploy Preview https://deploy-preview-122--palaviajeros.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Lighthouse
1 paths audited
Performance: 62
Accessibility: 86
Best Practices: 92
SEO: 85
PWA: -
View the detailed breakdown and full score reports

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