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

Issue #448 #497

Open
wants to merge 59 commits into
base: main
Choose a base branch
from
Open

Issue #448 #497

wants to merge 59 commits into from

Conversation

IgorMonardez
Copy link
Contributor

No description provided.

# Conflicts:
#	db/schema.rb
# Conflicts:
#	Gemfile
#	Gemfile.lock
#	db/schema.rb
@IgorMonardez IgorMonardez changed the title Issue 448 Issue #448 Sep 2, 2024
@IgorMonardez IgorMonardez linked an issue Sep 2, 2024 that may be closed by this pull request
…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.
@IgorMonardez IgorMonardez changed the title WIP: Issue #448 Issue #448 Sep 24, 2024
@JoaoFelipe
Copy link
Contributor

Encontrei 4 problemas na versão atual do PR:

1- Quando edito um Conceito Capes, um novo está sendo criado automaticamente. O novo só aparece depois de atualizar a página.

2- O Conceito Capes usado no Histórico está errado.

Tenho 2 conceitos cadastrados:

  • Conceito 5 de Setembro de 2014 a Outubro de 2024
  • Conceito 6 a partir de Outubro
    image

E uma matrícula com desligamento por titulação em Setembro de 2022:
image

No histórico está saindo com Conceito 6:
image

3- A validação de afiliações de professor está com problema. A seguinte situação deveria ser válida, mas está dando erro
image

4- Na exibição de professor com instituições, seria bom mostrar uma tabela com data de início e fim em cada uma delas, da mesma forma que tem a tabela de orientandos:
image


E uma sugestão de melhoria: a data de fim do Conceito Capes poderia estar com o horário padrão definido em 23:59:59 (veja como é feito em Disciplinas > Quadro de Horários). Mas isso não é uma mudança tão relevante se for dificil de fazer.

Copy link
Contributor

@JoaoFelipe JoaoFelipe left a comment

Choose a reason for hiding this comment

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

Coloquei comentários no código.

Além dos erros que marquei, aqui está dando erro no "olhinho" de matrícula:

NameError - undefined local variable or method `thesis_defense_date' for #<ActionView::Base:0x0000000097d588>:
  app/views/enrollments/_show_defense_committee_table.html.erb:15
  app/views/enrollments/_show_defense_committee_table.html.erb:10
  app/helpers/enrollments_helper.rb:261:in `enrollment_thesis_defense_committee_participations_show_column'

app/controllers/program_levels_controller.rb Outdated Show resolved Hide resolved
app/controllers/program_levels_controller.rb Outdated Show resolved Hide resolved
app/models/ability.rb Outdated Show resolved Hide resolved
spec/controllers/affiliation_controller_spec_spec.rb Outdated Show resolved Hide resolved
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.

Informações inconsistentes no histórico
3 participants