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

Ajusta para pegar os ids de série corretamente #145

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

gustavomendess
Copy link
Contributor

Descrição da mudança:

Por algum motivo, a tabela teacher_discipline_classrooms está sem os grade_ids, acredito que pode ser impacto da alteração das turmas multisseriadas.

Com isso, esta lógica que foi ajustada se fazia obsoleta.

Alteramos para, sempre pegar os grade_ids na tabela ClassroomsGrade, baseado no id da série(grade_id) e no id da turma(classroom_id) onde o professor tem alocação. Desta forma, a lógica deve funcionar sem maiores problemas.

Impactos da mudança:
Serão impactadas apenas as classes que dão include de TeacherRelationable e que, setam o atributo grades como uma coluna que deve ser validada.

Olhando todas as classes do projeto, a única que inclui a validação de série(grades) é a: i-diario/app/models/teaching_plan.rb, conforme print abaixo

image

Copy link
Member

@edersoares edersoares left a comment

Choose a reason for hiding this comment

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

Obrigado pela contribuição @gustavomendess, compreendi a solução.

A questão de grade_id null pode ser devido a um bug antigo na sincronização que criou novos registros com grade_id populado e não apgou estes registros.

@AnaPerola @Dmadaschi validem com qualidade por favor.

@edersoares edersoares changed the base branch from 1.5 to 1.6 June 19, 2024 18:52
@edersoares edersoares added ready and removed waiting labels Jun 20, 2024
@edersoares edersoares merged commit 29b7114 into portabilis:1.6 Jun 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants