PR de correção. Não precisa mergeear. #15
Conversation
components criados
layout do feed
Branch cris
rotas, login, baseUrl prontas
Tela inicio
Branch cris
Corrigi o css da tela de inicio e a chamada dele em router
Pronto Header e Footer, funcionais. ThemeProvider em App para ser usado
Trabalhando na página de cadastro e endereço
…la de login a revisar, e tela de cadastro de usuario em andamento
Tela Perfil em andamento, endereço e cadastro de endereço prontas, te…
corrigi o erro da página de cadastro de endereço, página de login con…
…, refatorei o login mas não consegui fazer funcionar, reeditei a tela de inicio, refatorei a tela de feed porem não terminei, minha turma sumiu, e eu me perdi nas datas, achei que hoje era sexta! fiz o maximo que pude sozinha !
Fiz a parte de cadastro e editar endereço, cadastro de usuario,perfil…
Search placehoder
| useEffect(() => { | ||
| /* axios.get('https://us-central1-missao-newton.cloudfunctions.net/futureEatsA/orders/history',{ | ||
| headers:{ | ||
| auth: props.token | ||
| } | ||
| }) | ||
| .then((res) => { | ||
| console.log(res.data) | ||
| }) | ||
| .catch((err) => { | ||
| }) */ | ||
|
|
||
| }) |
There was a problem hiding this comment.
Nunca se esqueçam de tirar os códigos comentados não usados. Sobretudo aqueles chamados dentro de um useEffect. Aqui, como vcs não colocaram nenhum array de dependências como segundo parâmetro do hook, isso está gerando um loop infinito de uma função que não faz nada.
| useEffect(() => { | ||
| axios.get('https://us-central1-missao-newton.cloudfunctions.net/futureEatsA/profile/address',{ | ||
| headers:{ | ||
| auth:token | ||
| } | ||
| }) | ||
| .then((res) => { | ||
| setAddress(res.data.address) | ||
| localStorage.setItem('address', JSON.stringify(res.data.address)) | ||
| }) | ||
| .catch((err) => { | ||
| console.log(err.message) | ||
| }) | ||
| },[]) |
There was a problem hiding this comment.
É sempre bom abstrair as lógicas de ciclo de vida e requisição em hooks e funções. Aqui, vcs poderiam criar um hook chamado useRequestData para fazer esse trecho, o que deixaria o componente mais limpo e reutilizável.
| const [street,onChangeStreet] = useInput(initalValue) | ||
| const [number,onChangeNumber] = useInput(initalValue) | ||
| const [neighbourhood,onChangeNeighbourhood] = useInput(initalValue) | ||
| const [city,onChangeCity] = useInput(initalValue) | ||
| const [state,onChangeState] = useInput(initalValue) | ||
| const [complement,onChangeComplement] = useInput(initalValue) |
There was a problem hiding this comment.
No lugar de um hook useInput, talvez fosse melhor usar um hook useForm, para economizar toda a lógica de input controlado em uma função só.
|
Oi, pessoal! Fiz uns comentários ao longo do código. Peço que deem uma olhada! Parabéns pelo trabalho e pelo esforço! |
PR de correção. Não precisa mergeear.