szekelymilan / e-kreta-cli

📚 Lightweight CLI app for e-Kréta.
https://www.npmjs.com/package/e-kreta-cli
MIT License
9 stars 1 forks source link

multi user kezeles #2

Open zoltansebestyen opened 4 years ago

zoltansebestyen commented 4 years ago

Jelenleg a user config a $HOME/.config/configstore/ mappaban az alabbi ket file-bol all:

Jo lenne egy megoldas, hogy tobb user (gyermek : ) ) esetén labellel (becenév) lehessen hivatkozni az egyes userekhez tartozó configra, azaz fenti fileok-ra, mindekettő kell minden egyes usernél.

szekelymilan commented 4 years ago

Valóban, hasznos funkció lenne. Majd megcsinálom. 😄

Gondolkodtam rajta, hogy a configstore helyett alkalmazhatnék valamilyen biztonságosabb megoldást - jelenleg a felhasználónév és a jelszó Base64-ben van tárolva, hogy legalább egy laikus ne tudja megszerezni a bejelentkezési adatokat, de valljuk be, ez nem profi megoldás. (Hashelés nem opció, a szerverre plain text formájában kell elküldeni a felhasználónevet és a jelszót.)

Egyébként az update-notifier-e-kreta-cli.json filet nem én hoztam létre, az update-notifier dependencyhez tartozik, ami az automatikus frissítésért felelős.

zoltansebestyen commented 4 years ago

Workaround a multiuser használtara: symlinket gyártottam a configstore és assignmetns mappákra, vsz. le is scriptelem 😄

szekelymilan commented 4 years ago

Valóban egy timestamp van az update-notifier-e-kreta-cli.json fileban, de azt az update-notifier dependency kezeli, nem tudom, pontosan hogyan működik. Tetszik ez a workaround, viszont meg lehetne rendesen is oldani, majd kibővítem ezzel a CLI-t. 😃

szekelymilan commented 4 years ago

Nem, nem ellenőrzi a kód a letöltött házi feladatokat, minden letöltésnél újra letölti az összeset. Nem tudom, hogy utólag lehet-e módosítani egy kiadott feladatot a KRÉTA rendszerben, de ha igen, akkor mindenképpen ez az előnyösebb megoldás - különben előfordulhatna, hogy a gépeden egy régi feladatverzió marad rajta. Azon gondolkodtam esetleg, hogy hasznos lenne-e, ha lenne egy státuszjelző funkció - tehát a házi feladatokat elvégzettnek (✔️) tudnád jelölni, ha megoldottad, és nem elvégzettnek (❌), ha még nem oldottad meg. Viszont itt is figyelni kéne arra, hogy esetleg változhat a házi feladat, és ha változott, azt szintén jelölni (❓) kell.

zoltansebestyen commented 4 years ago

Az fel sem merült bennem hogy a krétában lehet esetleg utólag módosítani a feladatokat, szerintem elég valószínűtlen, hogy a tanárok ilyen funkciót kihasználnak. Én a magam részéről a feladat állapotot amikor lesz címkézése vel oldom meg, azaz különböző színeket rendelek az egyes házifeladat fájlokhoz. Továbbá a feladat fájlok mappáját beraktam egy git repositoryba, így tudom követni ha esetleg mégis változtak volna.

Így nekem annyi bőven elég lenne, ha meglevő fájt a kréta kliens nem törölné le csak a tartalmát irná át.

szekelymilan commented 4 years ago

Nem törli le a feladatokat a kliens, csak felülírja a fileokat.

zoltansebestyen commented 4 years ago

Szuper, akkor tud annyit, amennyit én a Krétából ki szeretnék hozni 😄

zoltansebestyen commented 4 years ago

Ld. fent, lescripteltem a multi-user kezelést symlinkekkel, jól műxik.

szekelymilan commented 4 years ago

Rendes megoldást is szeretnék majd erre készíteni a jövőben, amihez nem kell egyénileg trükközni. 😄