Fix: persist filters on return #309
Conversation
src/pages/Home/Home.tsx
Outdated
| const storedFilterData = loadFilterData(); | ||
| const Keys = ['search', 'priority', 'cities']; |
There was a problem hiding this comment.
Como essa variável keys não sofre alterações, ela pode ser definida fora do render, não sendo recriada a todo momento.
Tenho a impressão que essa storedFilterData (que não é apenas dados armazenados) pode ser criada no início do componente e seu valor passado para o useState e reutilizada depois aqui.
There was a problem hiding this comment.
Vou realizar esses ajuste, válido demais seu comentário
src/pages/Home/Home.tsx
Outdated
|
|
||
| useEffect(() => { | ||
| saveFilterData(filterData); | ||
| }, [filterData]); |
There was a problem hiding this comment.
um pouco confuso storedFilterData e filterData referenciando a mesma coisa. eu removeria storedFilterData, passaria o loadFilterData() como argumento de useState em L37 e juntaria os 2 useEffect's
|
não seria mais pratico usar o context do que salvar no local ou vc criou algo para apagar depois? |
Adicione na função clearSearch para realizar a remoção |
|
Eu vou subir as alterações com base em todos os comentários, vlww |
|
Ajustado!! |
|
@Johnviti pode resolver o conflito, pf? |
Feito |
There was a problem hiding this comment.
primeiramente parabéns pelo trabalho @Johnviti
adicionei um minor e rodando localmente encontrei erros ao rodar:
npm run build:
npm run lint:
poderia dar uma olhada neles?
src/pages/Home/Home.tsx
Outdated
| }; | ||
|
|
||
| const loadFilterData = (): IFilterFormProps => { | ||
| const storedFilterData = JSON.parse(localStorage.getItem('filterData') || '{}'); |
There was a problem hiding this comment.
poderia mudar filterData pra kebab-case pra seguir o padrão local storage key names do projeto?
There was a problem hiding this comment.
Vou ajustar e verificar
|
Testado localmente e a funcionalidade atende os requisitos. O código atende e está apto a ser mergeado. |
5 participants
No description provided.