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

Refatora código e adiciona renderizador TSV #23

Merged
merged 47 commits into from
Jan 25, 2022
Merged

Refatora código e adiciona renderizador TSV #23

merged 47 commits into from
Jan 25, 2022

Conversation

rafaelpezzuto
Copy link
Contributor

@rafaelpezzuto rafaelpezzuto commented Jan 25, 2022

O que esse PR faz?

Este PR faz duas grandes implementações:

  1. Refatora código-fonte (remove, move, corrige código-fonte)
  2. Cria renderizador TSV para relatórios TR_J1, TR_J4, IR_A1 e CR_J1.

Onde a revisão poderia começar?

Por commits

Como este poderia ser testado manualmente?

  1. Instância a aplicação
  2. Para cada requisição efetuada, agrega ao final o parâmetro fmt=tsv.
  3. Observe que será baixado um arquivo em formato tsv como nos exemlpos apresentados nas figuras deste PR.

Algum cenário de contexto que queira dar?

Este PR faz parte uma série de melhorias a serem implementadas na SciELO SUSHI API.

Screenshots

Exemplo de relatório CR_J1 em formato TSV (requisição reports/cr_j1?begin_date=2019-01&end_date=2019-03&collection=scl&api=v2&fmt=tsv)
image

Exemplo de relatório TR_J4 em formato TSV (requisição reports/tr_j4?begin_date=2019-01&end_date=2019-03&collection=scl&fmt=tsv)
image

Exemplo de relatório IR_A1 em formato TSV (requisição reports/ir_a1?begin_date=2019-01&end_date=2019-03&collection=scl&api=v2&fmt=tsv&pid=)
image

Quais são tickets relevantes?

N/A

Referências

N/A

@rafaelpezzuto rafaelpezzuto merged commit 0c610d6 into scieloorg:master Jan 25, 2022
@rafaelpezzuto rafaelpezzuto deleted the refact-and-add-tsv branch January 25, 2022 00:09
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