Skip to content

Instantly share code, notes, and snippets.

@TeemuKoivisto
Last active June 6, 2017 08:44
Show Gist options
  • Save TeemuKoivisto/b52f789c8fd8743f616610400c396b45 to your computer and use it in GitHub Desktop.
Save TeemuKoivisto/b52f789c8fd8743f616610400c396b45 to your computer and use it in GitHub Desktop.
Miten grappaa voisi parantaa tekin osalta

Grappa checklist

  • Päivittää riippuvuudet ois kiva, webpack 1->2 rikkoo jotain niin jäi se siihen sillon joskus
  • Tai suoraan vaihtaa create-react-app:iin. Sillä pystyy helposti päivittämään riippuuvuksia
  • React koodi uudella rakenteella. Vanhakin on toiminut mutta ois ehk parempi laittaa tietyntyyppiset omiin kansioihin -> reducers, actions yms.
  • Vaihda API-middleware hässäkkä & redux-thunk redux-sagaan. On vaan parempi
  • Poista turha koodi. Heh
  • Poista/korjaa testit
  • Dokumentaatio
  • AUTOMAATTINEN BUILDI SUORAAN PALVELIMELLE. Aika must jos aiotaan käyttää Grappaa monessa paikkaa ja hallinnoida niitä päivityksiä jotenkin.
  • Sequelize vittuun ja natiivia SQL:ää tilalle. Oma suositus ainakin, on sen verran kökkö se api. Ja niin sqlitee ei sit vois enää käyttää lokaaliin devaukseen mut on se vaan parempi käyttää suoraan samaa tietokantaan ku sit käyttää tuotannossa. Ei tuu ikävii yllätyksii
  • Heittää vittuun / modifoida / mitä vaan / redux-form sille mun frontin validoinnille. Se tuli vaan tehtyy omasta tuskasta sen vaivaisen validoinnin kanssa ja redux-form oli vaan jotenkin niin tönkkö. Plus olis kiva vaan pastee samat validointi JSON:it bäkin puolelta fronttiin (se nyt oli se hyvä idea siinä et olis voinu jotenkin yhistää bäkin ja frontin validoinnit yhteen)
  • Flow vois olla hyvä jos haluaa vahvan tyypityksen. Ois niinku kätevää tai sen ainakin ite oon huomannu bäkin kaa temppuillessa
  • Päivitä koodi Node 8:n tasolle, async await ym vois olla kiva
  • PDF:ien generointia oikean kokoisina on yhtä helvettiä
  • Jotkut näistä kohdista voisi korjata https://securityheaders.io/?q=https%3A%2F%2Fgrappa.cs.helsinki.fi&followRedirects=on
  • Vaihda connect-busboy multeriin. Siinä kun on bugi.
  • PDF Abstract/Review näkymä frontin puolella on rikki muistaakseni Windows 7 FireFoxissa
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment