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

Solución al reto js-challenge #259

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Solución al reto js-challenge #259

wants to merge 10 commits into from

Conversation

Jhogakuse
Copy link

DESCRIPTION

Solución al reto: js-challenge

Nombre: Jhonatan Escalante
Usuario Platzi:Jhonatan Escalante
Correo Electronico: jhony.95.can@gmail.com

Reto:

  • Primer problema
  • Segundo problema
  • Tercer problema
  • Cuarto Problema
  • Quinto Problema

Jhogakuse and others added 10 commits June 9, 2022 19:03
- We get since the 5th product up to 10 products
- When we refresh the page we can see the next 10 products and when we are in the last page the pagination variable returns to the first one
- The logic for the infinite scroll was implemented  using an Intersection Observer
- Now the products are showed from the first one in groups of 10 elements 
- The local storage functionality was commented
- Now, the way to get all products is using the parameters from the API to get an specific limit and not all the products at once 
- A final message in the page when we get all products
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant