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

Tela de posters mostra alguns filmes repetidos #75

Open
guites opened this issue Sep 28, 2024 · 2 comments · May be fixed by #98
Open

Tela de posters mostra alguns filmes repetidos #75

guites opened this issue Sep 28, 2024 · 2 comments · May be fixed by #98
Labels
bug Something isn't working good first issue Good for newcomers website Ajustes e melhorias no portal

Comments

@guites
Copy link
Collaborator

guites commented Sep 28, 2024

Especialmente as URLs

Os filmes "lá nas matas tem", "way of giants" e "luz nos trópicos" estão repetindo várias vezes.

Eu acho que pode ser algum problema no query sendo feito no banco no endpoint que usamos pra buscar as imagens no banco.

O código do endpoint está aqui: flask_backend/routes/movie.py (função poster_images()), e a busca no banco acontece aqui: flask_backend/repository/movies.py (função get_paginated).

Detalhe: pra acessar as URLs listadas ali em cima, é preciso passar o cabeçalho X-LAZY-LOAD. Exemplo:

curl -H 'X-LAZY-LOAD: 1' "https://cinemaempoa.com.br/movies/posters/images?page=10"
@guites guites added bug Something isn't working good first issue Good for newcomers labels Sep 28, 2024
@guites
Copy link
Collaborator Author

guites commented Sep 28, 2024

@fbgrigolo eu fiz o merge do PR #54 mas notei que esse problema seguiu acontecendo

@guites guites added the website Ajustes e melhorias no portal label Oct 1, 2024
@guites
Copy link
Collaborator Author

guites commented Oct 8, 2024

@Gpocas

Puxando pra cá a discussão que começou em #90 , esse comentário trás uma situação em que os posters ainda estão aparecendo duplicados: #90 (comment) .

Para testar, usar esse dump do banco: https://drive.google.com/drive/folders/1c57i0LdFqWWt7QwdI7cLFQhgpMjuiqUE?usp=drive_link

@Gpocas Gpocas linked a pull request Oct 14, 2024 that will close this issue
@guites guites linked a pull request Oct 15, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers website Ajustes e melhorias no portal
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant