Group, Groupmembership

[peterlipt]

Issues #8 , #9

Szerintem megcsináltam mindent. Tesztelni sajnos nem tudtam, úgyhogy nem vagyok benne biztos, hogy minden úgy működik ahogyan kell.

[csiszaralex]

Ha VSCode-ot használsz, akkor a saját beállításaidnál be tudod állítani, hogy a mentésnél automatikusan sortolja az importokat a következő módon:

{
  "editor.codeActionsOnSave": {
      "source.fixAll": "explicit",
      "source.organizeImports": "explicit",
      "source.sortMembers": "explicit"
    },
}

Vagy tudod futtatni a backend mappában a következő parancsot kiadva:

npx eslint --fix

Amennyiben nem értenéd: az npx parancs a node-al automatikusan jön és arra képes, hogy az adott mappa node moduljait futtasd anélkül, hogy globálisan telepítened kéne őket, a --fix pedig "endedélyt ad rá", hogy ne csak kiírja, hanem javítsa is a hibákat.

U.i.: Ha az elsőt választod és be van kapcsolva az, hogy automatikusan mentse a fájlokat, ne CTRL+S-re, akkor nyomnod kell egy CTRL+S-t ahhoz, hogy rendezze az importokat. Valamint ez nem csak rendezi, hanem minden javítható hibát is javít és kitörli a nem használd importokat is.

[peterlipt]

Köszi már be is állítottam :))

[csiszaralex]

Emellett mi még teszteket nem tanultunk, ezt tényleg én nem mondtam el, de ugye a nest g res <név> az legenerálja a hozzá tartozó teszt fáljokat is (.spec.ts), ezt úgy lehet javítani, hogy nest g res <név> --no-spec parancsot adod ki.

[csiszaralex]

Hagytam pár (9) kommentet a kódban, ősszességében ami a feladat volt azt nagyon jól megcsináltad, egy pár apróbb hibád van, amiket a kódba kommenteltem.
Szép munka!

[peterlipt]

Köszi! Amint lesz időm ránézek és kijavítom.

[peterlipt]

Dolgoztam a group membership kezelésén, de az addMember metódust nem tudtam sehogy se működésre bírni.

[balintking]

Very nice job overall, only some minor remarks!👏

[balintking]

👌

[csiszaralex]

Amúgy jó, legközelebb kérlek figyelj arra, hogy yarn-t használunk, azaz yarn i-val telepítünk, nem npm i-vel és akkor nem lesz package-lock.json.