Skip to content
This repository was archived by the owner on Oct 17, 2024. It is now read-only.

Conversation

@snt-pedro
Copy link
Member

Feature ListarAcervo

Resolve

#36

Descrição

Cria a página de listagem do acervo, exibindo os itens públicos e privados, a depender da autenticação.

Adições

Dois novos data-cy ao CardItemAcervo (item-privado e item-publico) para facilitar os testes da página de Acervo.
Página mockada idêntica a do Acervo (a única diferença é o getItensAcervo, que retorna uma lista pronta pra não acessar o firebase). Necessário por limitações do cypress.

Alterações

A página de acervo agora deve carregar os últimos 6 itens em ordem decrescente por ordem cronológica, e não mais em ordem alfabética, por padrão.

Remoções

Verificando

Desenvolvedor

  • deve renderizar o loading e os itens em caso de sucesso
  • deve carregar os últimos 6 itens em ordem decrescente
  • deve mostrar apenas itens públicos quando não logado
  • deve mostrar itens privados quando logado
  • não deve repetir itens na mesma página

Revisor

  • Funcionalidade verificada localmente
  • Novos testes passam localmente e nos workflows

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

/>
<FormControl sx={{minWidth: '150px'}}>
<InputLabel>Coleção</InputLabel>
<Select
Copy link

Choose a reason for hiding this comment

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

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

/>
<FormControl sx={{minWidth: '150px'}}>
<InputLabel>Coleção</InputLabel>
<Select
Copy link

Choose a reason for hiding this comment

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

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

@Joao-Pedro-P-Holanda Joao-Pedro-P-Holanda merged commit 310f95c into main Sep 25, 2024
@Joao-Pedro-P-Holanda Joao-Pedro-P-Holanda deleted the feat/listar-acervo-fixed branch September 25, 2024 11:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants