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

Ajuste na recuperação de vínculo de alunos #105

Closed
wants to merge 32 commits into from

Conversation

williamsla
Copy link
Contributor

Ajustando a recuperação do vínculo de alunos na turma. Estava causando erro ao abrir avaliações descritivas

Descrição

Forçando a verificação de presença de vínculo de matrícula para os alunos

Contexto e motivação

Não estava abrindo avaliações descritivas. Ao tentar criar uma nova avaliação descritiva o sistema redirecionava para a página inicial e mostrava uma mensagem genérica de erro.

Issue #102. (#102)

Tipos de alterações

  • ✅ Correção de bugs (Não quebra outras funcionalidades)

Checklist:

  • ✅ Eu li o documento CONTRIBUTING. [REQUIRED]
  • ✅ Meu código segue o style guide. [REQUIRED]

Ajustando a recuperação do vínculo de alunos na turma. Estava causando erro ao abrir avaliações descritivas
Ajuste na recuperação de vínculo de alunos
Copy link
Contributor

@gustavomendess gustavomendess left a comment

Choose a reason for hiding this comment

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

Bom dia @williamsla, tudo bem?

Na realidade, aqui a gente contorna o problema e não o resolve de fato, o problema se dá pois existem dados inconsistentes, que seria uma enturmação: StudentEnrollmentClassroom, que está sem a matrícula: StudentEnrollment, em tese, esse problema deve ser resolvido quando o ambiente for sincronizado.

A gente tenta evitar safe navigation, pois corremos o risco de ficar com dados inconsistentes no banco que são contornados pelos mesmos.

@williamsla
Copy link
Contributor Author

williamsla commented May 24, 2022

Boa tarde @gustavomendess !

Entendi. Com a sincronização não resolveu, eu havia sincronizado várias vezes e continuava apresentando o problema, conferia no i-educar e os alunos da turma estavam matriculados corretamente, apareciam em todos os relatórios.

Então, pode ser que o problema esteja no componente de sincronização. Certo?

@gustavomendess
Copy link
Contributor

Boa tarde @gustavomendess !

Entendi. Com a sincronização não resolveu, eu havia sincronizado várias vezes e continuava apresentando o problema, conferia no i-educar e os alunos da turma estavam matriculados corretamente, apareciam em todos os relatórios.

Então, pode ser que o problema esteja no componente de sincronização. Certo?

Exatamente, pelo que vi agora, minha explicação acima, ficou só incorreta os nomes, o seu problema é que existe um StudentEnrollment que não tem um Student, que é um relacionamento que deve existir, então, você deve dar uma olhada no banco e entender o pq do Student vinculado ao StudentEnrollment não estar no sistema, e com isso, resolver o seu problema.

Caso seja uma falha de sync, por gentileza, me avisa! foi tratado recentemente um problema, mas o problema era que:
Uma enturmação StudentEnrollmentClassroom estava no banco com uma matrícula StudentEnrollment inativa, dai a solução, foi, no Synchronizer de enturmações, caso a matrícula esteja vazia, ele faz o soft delete da enturmação, para deixar o banco consistente.

Solução para o meu caso que citei:

image

no arquivo: student_enrollment_classroom_synchronizer.rb

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 2, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@edersoares edersoares changed the base branch from 1.4 to 1.5 December 14, 2023 22:55
@edersoares edersoares closed this Jun 19, 2024
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.

3 participants