-
Notifications
You must be signed in to change notification settings - Fork 14
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
Issue #448 #497
base: main
Are you sure you want to change the base?
Issue #448 #497
Conversation
…gh the table affiliation
…d adjustments on i18n
…of enrollments pdf
…m affiliation and program_level
…mitee_table and tests
# Conflicts: # db/schema.rb
# Conflicts: # Gemfile # Gemfile.lock # db/schema.rb
# Conflicts: # Gemfile
…rogram level" model scopes, changed tests and application of scopes in pdf and view
Mais alguns comentários. Começando com perguntas que acho que @leomurta pode ajudar a dar opinião:
Além disso, tenho algumas sugestões de mudanças:
Acho que seria melhor não ter esse link e as afiliações aparecerem no olhinho. |
…iation when there is an institution
A rota para o program level estaria na configuration? |
Eu acho que o ideal seria manter o comportamento atual.
Eu acho que se ele participou de alguma banca com aquela afiliação, não pode ser removido, pois quando tirarmos o histórico do aluno, não conseguiremos saber quem foi a banca dele. Mas o mero fato de ter afiliações sem estar vinculado a bancas, orientações, etc, não me parece relevante para barrar a remoção. Aí seria um cascade, que remove tanto o prof. quanto as afiliações (mantendo as instituições, obviamente).
Concordo.
Concordo. Tem que testar se o nível antes da migração (com a variável) é o mesmo depois da migração (com a tabela).
Concordo.
Aqui não sei se entendi bem. Não teria uma tela de ProgramLevel em Configurações onde podemos criar/editar/remover ProgramLevels? Na verdade, acho que são dois conceitos distintos: um é com as opções de nível (3 a 7). Talvez o nome seria ProgramLevelType para ficar parecido com o que fazemos no resto do Sapos. Outro com a atribuição de um nível ao programa numa data. Essa imagem não apareceu.
Talvez na edição do professor seja o melhor local para editar as afiliações.
Rota = menu, né? Acho que sim. Leo |
Não entendi, pode ter mais de um ProgramLevel que iriam de 3 a 7? Sobre o nome eu segui o mesmo que estava nos tipos das variáveis, acredito que não tenha por que mudar. |
Qual seria o comportamento atual? |
Hoje existe esses 5 níveis. Mas a CAPES pode decidir amanhã em trocar isso. O ideal seria isso ser flexível, como fazemos para vários outros dados, como tipo de matrícula, níveis, tipo de bolsa, etc. |
Tente fazer um teste com a versão 6.4.8 para identificar e documente aqui. |
…el at configuration
A rota em si seria /program_levels
Sim, é necessário ter isso. Não tinha quando revisei o PR.
Acho que criar ProgramLevelType seria complicação demais a toa. Não vejo problema em ter só ProgramLevel com datas do nível e um campo numérico (ou mesmo texto, se quiser deixar bem flexível caso a CAPES resolva mudar os níveis para letras no futuro). Se percebermos que ProgramLevel tem características a mais de acordo com nível, passa a fazer sentido ter o ProgramLevelType |
Beleza então. Talvez uma validação importante seja garantir que um nível não esteja com datas sobrepostas de validade, né? Deve ter uma validação parecida em bolsa. |
…ration screen, removal of the affiliation nested in teacher, in addition to the end and start dates in the program level show, update and create.
No description provided.