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: calengrade issues #250

Merged
merged 1 commit into from
Jan 5, 2024
Merged

fix: calengrade issues #250

merged 1 commit into from
Jan 5, 2024

Conversation

RenanLorijola
Copy link
Collaborator

Descrição

Corrigindo problemas das datas dos eventos gerados no nosso calengrade por conta de migração de bibliotecas e refatoração sem os devidos testes.

Tickets relacionados

#235

Como testar esse PR

Gere um arquivo ics no calengrade antigo e em seguida gere no nosso, baixe ambos os arquivos, execute um diff entre os arquivos ou abra-os com um editor de texto e cole num diff checker online (exemplo) e verifique que somente os campos de UID (id unico do evento) e timestamp (horário que o evento foi criado) tem valores diferentes. Também pode-se gerar o calengrade e colocar na sua agenda, verificando se os horários e dias estão corretos.

Screen.Recording.2023-12-28.at.01.34.29.mov

Adicionou/atualizou testes automatizados?

  • Sim
  • Não, porque: Aplicação ainda não tem testes configurados
  • Preciso de ajuda para escrever testes

@RenanLorijola RenanLorijola linked an issue Dec 28, 2023 that may be closed by this pull request
@Joabesv Joabesv added bug Something isn't working ready for review PR is ready for review labels Dec 28, 2023
@Joabesv Joabesv self-requested a review December 28, 2023 09:08
@RenanLorijola RenanLorijola changed the base branch from main to develop December 28, 2023 18:14
@brMonteiro-G brMonteiro-G self-requested a review January 5, 2024 03:20
Copy link
Member

@brMonteiro-G brMonteiro-G left a comment

Choose a reason for hiding this comment

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

LGTM

@brMonteiro-G brMonteiro-G merged commit 7668b82 into develop Jan 5, 2024
4 checks passed
@brMonteiro-G brMonteiro-G deleted the fix/calengrade-file branch January 5, 2024 03:21
@Joabesv Joabesv mentioned this pull request Jan 5, 2024
3 tasks
RenanLorijola added a commit that referenced this pull request Jan 5, 2024
mateusbrg added a commit that referenced this pull request Feb 16, 2024
* chore: update packages from root

* fix: facebook login (#194)

* chore: update dependencies (#201)

* chore: update dependencies

* chore: checkout actions v4

* fix: teachers form (#200)

* chore: adjust types and fix aliasInitials bugh

* draft: facebook fix

* refactor: auth route guard

* draft: facebook impl

* feat: better css

* fix: update index url

* fix: remove eslint disable and todo comment

* feat: relative path fb link

---------

Co-authored-by: Renan Zago Lorijola <[email protected]>
Co-authored-by: Mateus Braga <[email protected]>
Co-authored-by: RenanLorijola <[email protected]>

* fix: calengrade issues (#250)

* fix: facebook error handling

* refac: reorder stuff on script tag

* fix: 'home' to 'pagina inicial'

---------

Co-authored-by: RenanLorijola <[email protected]>
Co-authored-by: Joabe Silva <[email protected]>
Co-authored-by: Renan Zago Lorijola <[email protected]>
Co-authored-by: Joabesv <[email protected]>
RenanLorijola added a commit that referenced this pull request Feb 16, 2024
* chore: update packages from root

* fix: facebook login (#194)

* chore: update dependencies (#201)

* chore: update dependencies

* chore: checkout actions v4

* fix: teachers form (#200)

* chore: adjust types and fix aliasInitials bugh

* draft: facebook fix

* refactor: auth route guard

* draft: facebook impl

* feat: better css

* fix: update index url

* fix: remove eslint disable and todo comment

* feat: relative path fb link

---------

Co-authored-by: Renan Zago Lorijola <[email protected]>
Co-authored-by: Mateus Braga <[email protected]>
Co-authored-by: RenanLorijola <[email protected]>

* fix: calengrade issues (#250)

* fix: facebook error handling

* refac: reorder stuff on script tag

* fix: 'home' to 'pagina inicial'

---------

Co-authored-by: RenanLorijola <[email protected]>
Co-authored-by: Joabe Silva <[email protected]>
Co-authored-by: Renan Zago Lorijola <[email protected]>
Co-authored-by: Joabesv <[email protected]>
Joabesv added a commit that referenced this pull request Feb 16, 2024
* chore: update packages from root

* fix: facebook login (#194)

* chore: update dependencies (#201)

* chore: update dependencies

* chore: checkout actions v4

* fix: teachers form (#200)

* chore: adjust types and fix aliasInitials bugh

* draft: facebook fix

* refactor: auth route guard

* draft: facebook impl

* feat: better css

* fix: update index url

* fix: remove eslint disable and todo comment

* feat: relative path fb link

---------

Co-authored-by: Renan Zago Lorijola <[email protected]>
Co-authored-by: Mateus Braga <[email protected]>
Co-authored-by: RenanLorijola <[email protected]>

* fix: calengrade issues (#250)

* fix: facebook error handling

* refac: reorder stuff on script tag

* fix: 'home' to 'pagina inicial'

---------

Co-authored-by: RenanLorijola <[email protected]>
Co-authored-by: Joabe Silva <[email protected]>
Co-authored-by: Renan Zago Lorijola <[email protected]>
Co-authored-by: Joabesv <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready for review PR is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Geração de Calengrade
3 participants