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

Declarações customizáveis #508

Merged
merged 40 commits into from
Jan 15, 2025
Merged

Declarações customizáveis #508

merged 40 commits into from
Jan 15, 2025

Conversation

caiovelp
Copy link
Contributor

@caiovelp caiovelp commented Dec 9, 2024

Esse PR trata das issues

#218
#350
#217

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.

Ainda não executei para conferir, mas coloquei alguns comentários

spec/models/assertion_spec.rb Outdated Show resolved Hide resolved
config/locales/assertion.pt-BR.yml Outdated Show resolved Hide resolved
config/locales/pt-BR.yml Outdated Show resolved Hide resolved
@caiovelp caiovelp self-assigned this Jan 6, 2025
@JoaoFelipe
Copy link
Contributor

A geração de "Declaração de relatório de disciplinas para aluno avulso" está com um problema aqui:

image

O template parece certo:

DECLARAÇÃO

        Declaro, para os devidos fins, que <%= var('nome_aluno') %> cursou como Aluno Avulso as seguintes disciplinas do Programa de Pós-Graduação em Computação, nos termos do Art. 15 do Regulamento dos Programas de Pós-Graduação Stricto Sensu da Universidade Federal Fluminense.

        <% records.each do |record| %>
          <%= record['counter'] %>. Nome da disciplina: "<%= record['nome_disciplina'] %>"
          Carga horaria total: <%= record['carga_horaria'] %>
          Período: <%= var('ano_disciplina') %>/<%= var('semestre_disciplina') %>
          Nota: <%= record['nota'] %>
          Situação final: <%= record['situacao'] %>
        <% end %>

Fui investigar o código e parece que ao invés de usar o sistema de templates da notificação, as declarações foram implementadas com regex (função replace_placeholders de assertions_pdf_helpers.rb

Será que você não consegue usar o ERB formatter para deixar o sistema mais flexível que regex?

Se usar ERB formatter, não acho que vai ser necessário ter o record['counter']: ao invés disso, dá pra fazer a iteração com each_with_index.

@JoaoFelipe
Copy link
Contributor

Acho que seria interessante ter uma forma de definir qual a forma de assinatura por declaração, mas talvez seja melhor deixar estabilizar esse PR e deixar isso pra uma issue futura.

app/helpers/assertions_pdf_helper.rb Outdated Show resolved Hide resolved
app/helpers/assertions_pdf_helper.rb Outdated Show resolved Hide resolved
app/views/queries/_sql_form_column.html.erb Outdated Show resolved Hide resolved
@caiovelp caiovelp requested a review from JoaoFelipe January 8, 2025 03:43
@leomurta leomurta merged commit 162753d into develop Jan 15, 2025
2 checks passed
@caiovelp caiovelp deleted the issue-218 branch January 15, 2025 01:40
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