jounigit
commented
3 years ago Kiitos palautteesta!
Olen juuri tutustumassa ja treenaamassa docker:ia. Olen jo tässä vaiheessa vaikuttunut - eka tutorial takana.
Tuon monorepo ongelmankin huomasin vasta loppusuoralla.
Testasin kuvalatauksen ja kaikki toimi moitteetta tällä kertaa ja edellisilläkin kerroilla. Myös kavereiden testailut ovat toimineet.
Kuvalataus cloudinariin voisi myös tapahtua jo frontista. En tosin tiedä vielä nopeuttaisiko se latausta.
ke 17. maalisk. 2021 klo 17.50 Jami Kousa @.***) kirjoitti:
Full Stack projektin katselmointi
Tässä lyhyt katselmointi Full Stack kurssin projektistasi. Katselmoinnissa esitetyt kommentit ovat parannusehdotuksia, joita voit halutessasi ottaa huomioon tässä tai tulevissa projekteissasi. Mitään muutoksia tähän projektiin ei siis ole pakko tehdä suoritusmerkintää varten. Käytettävyys Mitä tein?
Rekisteröidyin
Kirjauduin
Lisäsin kuvan
Klikkailin ympäriinsä Kokemus
Aika hyvän näköinen
Kuvaa lisätessä ei oikeastaan mitään tapahtunut ja sivu ei toiminut uudelleenpäivityksen jälkeen Koodi
Koodi hyvälaatuista, jaettu selkeisiin kokonaisuuksiin ja tiedostot löytyvät oikeilta paikoilta. Hyvää työtä!
git on parempi versionhallinnan työkalu kuin kommentit https://github.com/jounigit/ja-galleria-frontend-2/blob/25a94e4677f152fa1a9e603ea4ea0738ded23e46/src/components/Lightbox/Lightbox.js#L4
Minulla on hyvin vähän mitään valitettavaa tai parannusideoita koodin puolelle. Hienoa!
Keksin väkisin jotain kerrottavaa, niin kannattaa tutustua muutamaan käsitteeseen joista voi tulevaisuudessa olla iloa:
Continuous Integration / Trunk Based Development eli hylätään nuo feature branchit: https://trunkbaseddevelopment.com/
Monorepo eli siirretään bäkki ja frontti samaan repoon koska miksi ne edes ovat erillään: https://trunkbaseddevelopment.com/monorepos/
Valmiin projektin voi paketoida julkaisua varten docker konttiin: https://www.docker.com/, allekirjoittaneella kurssi tästä aiheesta https://devopswithdocker.com Kokonaisuus
Koodilaatu on esimerkillistä. Ehkä jopa hieman liikaakin hiottua! Selvittele tuo päivittämisen bugi ja käytä vielä aikaa käyttöliittymän hiomisessa niin tästä tulee timanttinen portfolio projekti CVhen alkuun.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jounigit/ja-galleria-frontend-2/issues/4, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJCP3JDOM2TDPRICLLNRLFTTEDFURANCNFSM4ZK3GGXQ .
Full Stack projektin katselmointi
Tässä lyhyt katselmointi Full Stack kurssin projektistasi. Katselmoinnissa esitetyt kommentit ovat parannusehdotuksia, joita voit halutessasi ottaa huomioon tässä tai tulevissa projekteissasi. Mitään muutoksia tähän projektiin ei siis ole pakko tehdä suoritusmerkintää varten.
Käytettävyys
Mitä tein?
Rekisteröidyin
Kirjauduin
Lisäsin kuvan
Klikkailin ympäriinsä
Kokemus
Aika hyvän näköinen
Kuvaa lisätessä ei oikeastaan mitään tapahtunut ja sivu ei toiminut uudelleenpäivityksen jälkeen
Koodi
Koodi hyvälaatuista, jaettu selkeisiin kokonaisuuksiin ja tiedostot löytyvät oikeilta paikoilta. Hyvää työtä!
git on parempi versionhallinnan työkalu kuin kommentit https://github.com/jounigit/ja-galleria-frontend-2/blob/25a94e4677f152fa1a9e603ea4ea0738ded23e46/src/components/Lightbox/Lightbox.js#L4
Minulla on hyvin vähän mitään valitettavaa tai parannusideoita koodin puolelle. Hienoa!
Keksin väkisin jotain kerrottavaa, niin kannattaa tutustua muutamaan käsitteeseen joista voi tulevaisuudessa olla iloa:
Continuous Integration / Trunk Based Development eli hylätään nuo feature branchit: https://trunkbaseddevelopment.com/
Monorepo eli siirretään bäkki ja frontti samaan repoon koska miksi ne edes ovat erillään: https://trunkbaseddevelopment.com/monorepos/
Valmiin projektin voi paketoida julkaisua varten docker konttiin: https://www.docker.com/, allekirjoittaneella kurssi tästä aiheesta https://devopswithdocker.com
Kokonaisuus
Koodilaatu on esimerkillistä. Ehkä jopa hieman liikaakin hiottua! Selvittele tuo päivittämisen bugi ja käytä vielä aikaa käyttöliittymän hiomisessa niin tästä tulee timanttinen portfolio projekti CVhen alkuun.