Facebook
From Chocolate Parrot, 3 Years ago, written in Plain Text.
Embed
Download Paste or View Raw
Hits: 72
  1. https://gist.github.com/HubDoggo/dec55f6ca86d2536466a1270f36a0496#file-client-js-L21
  2. Ei välttämättä ole loading, saisi olla statessa loading täppä.
  3. https://gist.github.com/HubDoggo/dec55f6ca86d2536466a1270f36a0496#file-client-js-L59
  4. error handling missing
  5. https://gist.github.com/HubDoggo/dec55f6ca86d2536466a1270f36a0496#file-client-js-L64
  6. excessive usage of ternary, please make an input validation function to validate this data
  7. https://gist.github.com/HubDoggo/dec55f6ca86d2536466a1270f36a0496#file-client-js-L68
  8. ei pistetä logiikkaa näin paljaasti jsx:ään, tehdään handleri
  9. https://gist.github.com/HubDoggo/dec55f6ca86d2536466a1270f36a0496#file-client-js-L68
  10. spaghetti begins. Tän sais heittää moneks pienemmäks metodiks
  11. https://gist.github.com/HubDoggo/dec55f6ca86d2536466a1270f36a0496#file-client-js-L102
  12. jestas, tämmöset dynaamiset hommelit generoidaan ennen renderiä
  13.  
  14.  
  15. https://gist.github.com/HubDoggo/dec55f6ca86d2536466a1270f36a0496#file-express-server-js-L1
  16. oh my, ei käytä ES6. Babel käyttöön oitis!
  17. https://gist.github.com/HubDoggo/dec55f6ca86d2536466a1270f36a0496#file-express-server-js-L18
  18. ei kovakoodata näin, conffi tiedostoon pls
  19. https://gist.github.com/HubDoggo/dec55f6ca86d2536466a1270f36a0496#file-express-server-js-L48
  20. kunnon loggeri käyttöön, esim winston
  21.