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

Feat/food request nham connection #777

Open
wants to merge 50 commits into
base: dev
Choose a base branch
from

Conversation

Gusta-Santos
Copy link
Collaborator

@Gusta-Santos Gusta-Santos commented Jul 15, 2024

Motivação

Para complementar o processo da merenda escolar, modificações foram feitas em 3 partes do módulo de merenda, editais, agricultores e solicitações. As modificações realizadas nessas telas foram feitas com a finalidade de aperfeiçoar o processo, de acordo com o solicitado pelas nutricionistas, e estabelecer a conexão com o aplicativo do NHAM para os agricultores.

Alterações Realizadas

  • Modificações nos cadastros de editais e agricultores.
  • Permitindo o envio de PDF para o aplicativo do agricultor ao cadastrar ou alterar um edital.
  • Modificações no processo de gerar uma solicitação, adicionadas as possibilidades de estabelecer a relação entre uma solicitação e várias escolas e vários agricultores.
  • Adicionado um modal para exibir os detalhes de uma solicitação.
  • Adicionada a tela de ativar e inativar tanto editais como agricultores.

Fluxo de Teste

Antes de iniciar o fluxo de testes é necessário observar se o novo módulo de merenda escolar está ativo.

🧪 Fluxo de Teste 01: Cadastrar e alterar edital.

É necessário observar se o novo módulo de merenda está ativado.

Parte 1 - Cadastro
- Acessar a parte de editais no novo módulo de merenda
- Acessar a tela de cadastro de edital
- Preencher as informações das sessões "Informações do edital" e "Itens do edital", após preenchidas, salve o edital.

Parte 2 - Alterações
- Acessar o edital que acabou de ser criado e clicar no botão de "visualizar edital", verificiar se o pdf que foi inserido previamente abre em uma outra guia.
- Alterar as informações do edital criado e verificar estão sendo salvas corretamente.

Parte 3 - Mudanças estado ativo ou inativo
- Clicar no botão da lixeirinha para inativar um edital
- Observar se o mesmo foi retirado da listagem inicial de editais
- Clicar no botão exibir inativos e verificar se o edital recém inativado está sendo exibido lá como inativado
- Clicar no botão que fica abaixo da sessão "ativar/inativar" no respectivo edital inativado para ativá-lo novamente.
- Retornar para a parte de editais e verificar se o edital volta a aparecer.

🧪 Fluxo de Teste 02: Cadastrar e alterar agricultores.

Parte 1 - Cadastro
- Acessar a parte cadastro de agricultores no novo módulo de merenda
- Informar um CPF, para que possa ocorrer a busca no aplicativo de agricultores e verificar se lá existe algum agricultor cadastrado, se sim, os campos bloqueados serão preenchidos com as informações do aplicativo, se não, os campos bloqueados ficaram livres para que possam ser preenchidos.
- Preencher as sessões "Dados básicos" e "Relações de produtos" e salvar o agricultor.

Parte 2 - Alterações
- Alterar as informações do agricultor cadastrado e verificar estão sendo salvas corretamente.

Parte 3 - Mudanças estado ativo ou inativo
- Realizar os mesmos passos descritos para ativar ou inativar um edital, mas dessa vez para agricultores.

🧪 Fluxo de Teste 03: Gerar solicitação

- Acessar a parte solicitações no novo módulo de merenda
- Clicar em gerar solicitação
- Informar o edital no qual aquela solicitação é correspondente, as escolas que receberão os alimentos e os agricultores que serão responsáveis por cumprir a solicitação.
- Informar a relação dos produtos referentes a solicitação
- Salvar
- Clicar no ícone de lupa e observar se o modal aberto exibe as informações detalhas da solicitação gerada.

Migrations Utilizadas

2024-19-07_fix_food_notice_farmer_and_request

⚠️ Observações: ⚠️
Primeiro rode foodNotice.sql, depois farmerRegister.sql e depois foodRequest.sql

Checklist de revisão

  • O número da versão foi alterado no arquivo config.php?
  • Foi adicionada uma descrição das alterações no arquivo de CHANGELOG?
  • O pull request passou na avaliação do SonarLint?
  • O pull request está nomeado corretamente seguindo o padrão de nomes de branchs?

TI GUSTAVO and others added 30 commits April 2, 2024 10:16
…cionando a categoria e a imageUrl no envio da solicitação
Copy link
Collaborator

@AnotherOne07 AnotherOne07 left a comment

Choose a reason for hiding this comment

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

💡Sugestão de Melhoria:

  • Na tela de editar informações de edital é possível observar um delay no carregamento das informações, mesmo rodando a aplicação em localhost, tendo em vista a operação do sistema em produção, onde haverá um variação de hardware e qualidade de conexão com a internet, acredito que seria adequado, uma mudança que adicionasse um frame de "Loading" até que as informações estejam disponíveis para serem renderizadas na tela.

Copy link
Collaborator

@AnotherOne07 AnotherOne07 left a comment

Choose a reason for hiding this comment

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

ℹ️ Sugestão de Correção:

  • Visualização no mobile precisa ser corrigida, foram identificados os seguintes problemas:
  • Alguns campos estão desalinhados e com um tamanho que não se ajusta muito bem ao layout, sugiro verificar essas questões.

Copy link
Collaborator

@AnotherOne07 AnotherOne07 left a comment

Choose a reason for hiding this comment

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

💡Sugestão de Correção:

Arquivos SQL devem ser corrigidos:

  • Adicionar created_at e updated_at as tabelas onde está faltando essa propriedade.
  • Sugiro também que baixe uma versão atualizada do banco de um município e tente rodar os comandos SQL da migration, para verificar se estão funcionando adequadamente.

Copy link
Collaborator

@GabrielChagas14 GabrielChagas14 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

sonarcloud bot commented Sep 10, 2024

Copy link
Collaborator

@AnotherOne07 AnotherOne07 left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pronto para merge reviews feitas e aprovadas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants