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

fix: first day of month not finding sessions #113

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

KozielGPC
Copy link
Contributor

Descrição

Conforme indicado na issue #64, no primeiro dia do mês não era encontradas as sessões pois a data vinha como "1º de Maio", por exemplo. Para resolver esse problema, foi adicionado um replace na string da data removendo o "º' e substituindo por um espaço vazio, fazendo com que as sessões sejam encontradas corretamente

Screenshots

Antes da modificação - Exemplo 25/10
image

Após a modificação - Exemplo 25/10
image

Teste

Para testar a modificação, alterei manualmente o arquivo grade.html, adicionando o caractere especial na data:
image

@KozielGPC KozielGPC requested review from guites and a team as code owners October 26, 2024 02:53
@guites
Copy link
Collaborator

guites commented Oct 30, 2024

só passando pra agradecer pela contribuição, eu tô sem tempo essa semana mas vou tentar revisar o PR amanhã de noite, pra aproveitar e já usar ele no dia 01/11 =P

@KozielGPC
Copy link
Contributor Author

Tranquilo hahah, beleza, fico no aguardo então. Valeu!!

@KozielGPC
Copy link
Contributor Author

Fala @guites, tranquilo?! Será que você conseguiria revisar hoje esse PR para já validarmos o funcionamento no primeiro dia do mês? hahah

@guites
Copy link
Collaborator

guites commented Nov 1, 2024

Fala @guites, tranquilo?! Será que você conseguiria revisar hoje esse PR para já validarmos o funcionamento no primeiro dia do mês? hahah

Salve, bom dia! Então, eu acordei hoje determinado a revisar esse PR, mas fui surpreendido por um bug na pipe do Paulo Amorim (#117), e optei por fazer um quickfix pro site não ficar vazio! :(

Infelizmente (ou felizmente??) nessa nova grade semanal, eles não usaram o simbolozinho de primeiro (https://www.cinematecapauloamorim.com.br/grade-semanal) . Mas eu vou testar o seu PR com o html de maio, quando o problema aconteceu pela primeira vez.

Assim já ficamos blindados caso eles utilizem esse formato novamente.

@KozielGPC
Copy link
Contributor Author

Beleza! Fico no aguardo então

@KozielGPC
Copy link
Contributor Author

Fala @guites , tranquilo?! Você conseguiu dar mais uma olhada nesse PR aqui?

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.

2 participants