Skip to content

Instantly share code, notes, and snippets.

@brunokrebs
Created June 16, 2019 09:55
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save brunokrebs/cd6c732fc2a2671628d760705540c0f6 to your computer and use it in GitHub Desktop.
Save brunokrebs/cd6c732fc2a2671628d760705540c0f6 to your computer and use it in GitHub Desktop.
  1. Você não precisa gerenciar CORS na API. Para o ambiente de desenvolvimento você pode utilizar o React Proxy: https://facebook.github.io/create-react-app/docs/proxying-api-requests-in-development

  2. Ao utilizar o proxy informado acima, você vai poder remover http://localhost:3001 do seu código.

  3. Não precisa adicionar o termo fetch nas suas sagas. Ou seja, ao invés de fetchAddBuilding e callFetchAddBuilding, basta addBuilding e callAddBuilding.

  4. store.dispatch(loadBuildingsList()); não deveria estar no index.js (eu sei que no meu artigo está assim, mas aquilo é só um exemplo básico). Não são todas as páginas que vão precisar carregar a lista de buildings. Por exemplo, se o usuário for diretamente para http://localhost:3000/new-item, não tem porque disparar uma requisição para carregar a listagem de buildings.

  5. Inclusive, o fato de esse store.dispatch(loadBuildingsList()); estar no index faz com que ele seja chamado apenas uma vez. Se eu acesso o sistema, vou para a página de adicionar buildings, salvo um novo building, e volto para a página de listagem, meu novo building não aparece lá.

  6. Ao invés de listar as propriedades de cada building como uma lista de itens, seria melhor usar tabelas: https://www.w3schools.com/html/html_tables.asp

  7. Em relação ao loop infinito. Eu gostaria de saber se você sabe o porque ele acontece. Gostaria que você e explicasse, e depois vou lhe explicar como eu acho que vamos resolver. (obs., o que causa o problema eu já tenho certeza do que é, a solução ainda não tenho tanta certeza, mas acho que sei).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment