Skip to content

fix: Correção de visualização de dados no gráfico#35

Open
wgeovanni wants to merge 2 commits intomainfrom
24-willyan-04
Open

fix: Correção de visualização de dados no gráfico#35
wgeovanni wants to merge 2 commits intomainfrom
24-willyan-04

Conversation

@wgeovanni
Copy link
Contributor

O que foi feito?

Foi adicionado uma mensagem informando que não há dados para serem visualizados caso não houverem lançamentos. Realizada melhorias no design para se adequar ao exemplo do figma.
Agora o gráfico mostra a soma total de energia mensal gerado nos últimos 12 meses das unidades.

Como testar?

Inicialmente rode os seguintes comandos no terminal:
npm run server para rodar o app
npm run dev para rodar o servidor com os dados json
No arquivo json, adicione novos lançamentos para visualizar os dados no gráfico. Exemplo de dados que podem ser inseridos:
, {
"id_unidade": "def",
"data": "2022-08",
"total": 120,
"id": "3iTmfhJ"
},
{
"id_unidade": "def",
"data": "2022-06",
"total": 170,
"id": "3iTmfhJ"
},
{
"id_unidade": "def",
"data": "2022-10",
"total": 110,
"id": "3iTmfhJ"
},
{
"id_unidade": "def",
"data": "2022-07",
"total": 190,
"id": "3iTmfhJ"
},
{
"id_unidade": "def",
"data": "2022-09",
"total": 80,
"id": "3iTmfhJ"
},
{
"id_unidade": "def",
"data": "2023-11",
"total": 50,
"id": "3iTmfhJ"
},
{
"id_unidade": "def",
"data": "2023-01",
"total": 200,
"id": "3iTmfhJ"
}
Também pode ser feito o teste de apagar os dados de lançamentos para ser possível ver a mensagem de aviso de que não há lançamentos.
Pode também encerrar o server json para ver o popup de alerta.

Resultado Esperado?

Imagem prevista para caso não hajam lançamentos:
image

Imagem prevista para os dados de lançamentos do projeto mais os dados de exemplo:
image

Imagem do popup de alerta caso não haja conexão com o servidor:
image

@EricoCoutoJr
Copy link
Contributor

OI Willian. Poderia sujerir o valor zerado ao invés de NaN.... ficaria mais agradável para o usuário. O NaN não significa muita coisa para o usuário final.

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.

2 participants