Skip to content

feat: allow an undefined TEA support level when registering a student - #700

Closed
ErickMascarenhas wants to merge 1 commit into
refactor/settings-remove-tutorial-and-languagefrom
feat/support-level-undefined
Closed

feat: allow an undefined TEA support level when registering a student#700
ErickMascarenhas wants to merge 1 commit into
refactor/settings-remove-tutorial-and-languagefrom
feat/support-level-undefined

Conversation

@ErickMascarenhas

Copy link
Copy Markdown
Collaborator

@

Summary

  • Adicionou o valor indefinido ao ENUM nivel_suporte_tea por migration aditiva: nenhum registro existente é alterado e os três níveis continuam válidos. Como alunos.nivel_suporte é NOT NULL, "ainda não definido" precisava ser um valor do próprio tipo, e não NULL.
  • Incluiu a opção "Indefinido" no seletor de nível de suporte do cadastro/edição de aluno.
  • Centralizou a conversão rótulo -> código em toSupportLevelCode() (features/students/utils/support-level.ts), substituindo as duas implementações duplicadas em use-students e use-team-data. O nível é identificado pelo dígito, única parte comum a todas as grafias e ao código gravado; sem dígito, o resultado é indefinido — que agora também é o destino correto de um valor vazio ou desconhecido, antes silenciosamente convertido em nivel_1.
  • Passou a exibir "Indefinido" em todos os pontos que traduzem o nível: lista de alunos, perfil do aluno em sessões, análises, tela de relatório e exportações PDF/CSV.
  • Adicionou as chaves students.form.supportOptionUndefined, students.support.undefined e reports.supportLevelUndefined em pt e en.

Validation:

  • npx tsc --noEmit — 0 erros.
  • npm run lint — 0 erros (7 warnings pré-existentes, nenhum novo).
  • npx typedoc@0.28.19 — gerado sem avisos.
  • Varredura por nivel_1|nivel_2|nivel_3 em app/, components/, features/ e lib/ para garantir que todo mapeador de nível de suporte trata o novo valor.
  • supabase db lint não pôde ser executado localmente (Docker indisponível nesta máquina); a migration é validada pelo CI do PR.

Ordem de merge: depende do PR #699 (refactor/settings-remove-tutorial-and-language), no qual está empilhado por também tocar translations.ts. Este PR aponta para aquela branch; após o merge dele, o GitHub retarget para develop.
@

@ErickMascarenhas
ErickMascarenhas requested review from a team August 5, 2026 20:02
@ErickMascarenhas
ErickMascarenhas changed the base branch from refactor/settings-remove-tutorial-and-language to develop August 6, 2026 00:14
@ErickMascarenhas
ErickMascarenhas requested review from a team August 6, 2026 00:14
@ErickMascarenhas
ErickMascarenhas changed the base branch from develop to refactor/settings-remove-tutorial-and-language August 6, 2026 00:15
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.

1 participant