Skip to content
New issue

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

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

Already on GitHub? # to your account

Grupo 6_SkylabFilms #696

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

Grupo 6_SkylabFilms #696

wants to merge 140 commits into from

Conversation

blopez-dev
Copy link
Contributor

No description provided.

Santiago.Martinez and others added 30 commits May 3, 2021 15:43
gteixi
gteixi previously approved these changes May 11, 2021
Copy link
Contributor

@gteixi gteixi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

buen trabajo!!!!

upcoming: 'upcoming'
};

export const getFilms = (entryPoint) => http.get(`/movie/${entryPoint}?api_key=${API_KEY}&language=en-US&page=1`);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

podrías hacer una función con el listado de exportacines y aquí llamar a esa función para refactorizar este fichero?

IsabelGomez87
IsabelGomez87 previously approved these changes May 11, 2021
<GlobalApp>
<Auth0Provider
domain="dev-skylab.eu.auth0.com"
clientId="NtRILJGUuayWtHP9G8fFFDWgTFNMubaY"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Intentar no subir el clientId visible

@blopez-dev blopez-dev dismissed stale reviews from IsabelGomez87 and gteixi via ea9beeb May 12, 2021 14:38
Copy link
Contributor

@Laura-SM Laura-SM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!

@SantiagoMC52 SantiagoMC52 self-requested a review May 18, 2021 07:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants