Skip to content

Fix/class_contents_count#1008

Open
AnotherOne07 wants to merge 30 commits into
devfrom
fix/class_contents_count
Open

Fix/class_contents_count#1008
AnotherOne07 wants to merge 30 commits into
devfrom
fix/class_contents_count

Conversation

@AnotherOne07

@AnotherOne07 AnotherOne07 commented Jun 2, 2025

Copy link
Copy Markdown
Collaborator

🚀 Motivação

  • Usuários do TAG relataram que o relatório de "Ficha de Notas" estava apresentando informações inconsistentes para diferentes alunos em uma turma. Para correção foi proposto corrigir o cálculo da distribuição de carga horária entre as unidades, agora sendo baseado nos itens cadastrados na tabela de matriz curricular.

🔧 Alterações Realizadas

  • Função responsável por calcular a carga horário por unidade alterada para considerar a matriz curricular.
  • Remoção da linha "Total de aulas dadas" do relatório de "Ficha de Notas".

📌 Requisitos

  • Dump atualizado de um município.
  • Branch atualizada

🛠️ Fluxo de Teste

🧪 Fluxo de Teste 1 (FT1): Verificar relatório de ficha de notas

1. Acessar tela de turmas, acessar uma turma com pelo menos mais de um aluno matriculado.
2. Navegar até a aba de alunos, acesse dois alunos matriculados.
3. Navegar até a sessão de matrícula, expandir o acordeon de matrícula da turma selecionada e gerar o relatório
de "Ficha de Notas".
4. Verificar a sessão de carga horária dos dois alunos e verificar se os dados estão consistentes.

✅ Sucesso: Dados consistentes
❌ Falha: Dados inconsistentes

✨ Migrations Utilizadas

  • Nenhuma migration.

✔️ Checklist - Padrões para PR

  • O número da versão foi alterado no arquivo config.php?
  • Foi adicionada uma descrição das alterações no arquivo de CHANGELOG?
  • O pull request passou na avaliação do SonarLint?
  • O pull request está nomeado corretamente seguindo o padrão de nomes de branchs?

Documentação

Houve alteração nos fluxo de uso?
(Lembrete: Em caso afirmativo, adicionar label Atualização de manual)

  • Sim
  • Não

jfelipesouza and others added 29 commits May 16, 2025 14:11
* relise/v3.2.0

* v3.2.0

* feat: fixing register 20 to 2025

* adding script files

* chore: adding special education

---------

Co-authored-by: gabriel-IPTI <ti.gabriel@ipti.org.br>
Co-authored-by: GabrielChagas14 <84946326+GabrielChagas14@users.noreply.github.com>
* relise/v3.2.0

* v3.2.0

* chore: update record 00 document related to school_identification

* feat: fixing register 20 to 2025

* adding script files

* chore: adding special education

* feat: update

* feat:create field

* feat: update form

* add migration

* update

* feat: add sql code register 10

---------

Co-authored-by: gabriel-IPTI <ti.gabriel@ipti.org.br>
Co-authored-by: GabrielChagas14 <84946326+GabrielChagas14@users.noreply.github.com>
Co-authored-by: Igor Gonçalves <igor_goncalves@live.com>
@sonarqubecloud

sonarqubecloud Bot commented Jun 6, 2025

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ B)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@GabrielChagas14 GabrielChagas14 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ao realizar o teste com um dump de muribeca notei que a carga horária permaneceu a mesma para todas as unidades (segue foto), mesmo com as unidades tendo quantidades de dias letivos substancialmente diferentes segue foto dessa parte do relatório de antes e depois da alteração:

antes:
image

depois:
image

@jfelipesouza

Copy link
Copy Markdown
Collaborator

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants