cesko-digital / pomahejukrajine-web

https://www.pomahejukrajine.cz
MIT License
5 stars 6 forks source link

Dobrovolník (nabízející pomoc) může sám přidat ve svém profilu novou nabídku #26

Closed ZdenekMen closed 1 year ago

ZdenekMen commented 2 years ago

Dobrovolník po zavolání Moje nabídky na Front Page Portalu PU se dostane na obrazovku Moje nabídky, kde lze editovat již existující nabídky, A NOVĚ může přidávat pod svým jménem další nabídky, tj stejně jako by začínal z menu Nabízím pomoc - tj zaklikne jednu z možností a otevřou se mu další otázky k vyplnění image

image image

ZdenekMen commented 2 years ago

Ahoj Tomasi. @tgds prosím tě, budeš se tímto Issue zabývat? Je už tam 26 dnů. Pokud ne, mohl by se na něj třeba vrhnout někdo jiný. Mám pocit od @Michal-Miky-Jankovsky , že data by tam pro to už mohla být. Dej prosím vědět. Děkuji

Michal-Miky-Jankovsky commented 2 years ago

Podívám se večer

ZdenekMen commented 2 years ago

Přidávám doplnění pro zobrazení první vstupní obrazovky Moje nabídky pomoci: Při zobrazování nabídek jednoho dobrovolníka prosím zobrazte na dlaždici i Stav Nabídky. Pokud by se povedlo to odlišit barevně (text nebo celá dlaždice) tak prosím 3 kategorie

Číselník Stavů a jejich Typů viz ukázka z definice číselníku image

martinsik commented 2 years ago

Mě tady napadla ještě jedna věc. Když jsem přihlášený, tak link "Nabízím pomoc" na homepagi by měl asi vést na tu novou stránky s přidáním nabídky a ne na registraci.

Screenshot 2022-05-16 131050

matej-sveda commented 2 years ago

ten registracni formular je prave i formular pro novou nabidku, na te strance moje-nabidky ho pouzivam taky, akorat tam skryju ty zname pole o uzivateli.... ale muzemem to udelat, ze tlacitko "Nabizim pomoc", kter zminujes, povede na moje-nabidky, a pokud tam jeste nebude zadna existovat (neexistujici uzivatel), dostane se na reg. formular

matej-sveda commented 2 years ago

ale souasne to funguje tak, ze pokud se dostanu na moje-nabidky a neexistuje uzivatel, uz to redirectuje na /login, takze scenar z predchoziho komentare nedava smysl... ale jeste je moznost nastavit ten redirect na strance /nabidka (kam vede tlacitko "Nabiim pomoc")... pokud zrovna bude prihlaseny uzivatel, presmeruje ho to rovnou na /moje-nabidky, kde muze pridat dalsi nabidku... to mi prijde dobre :+1: co si myslis @ZdenekMen , chapeme se?

ZdenekMen commented 2 years ago

Ahoj pánové @matej-sveda @martinsik , koukám, že jste se do toho pěkně ponořili. A objevujete ty naše nelogizmy ;-). Mateji, máš asi pravdu, bylo by to lepší, ale já bych to držel, tak jak to je dnes tj i s určitou duplicitou.

Jedním z důvodů je i čas a vaše a moje omezená kapacita. A když Vám řeknu, že jako vedoucí projektu jsme měl přezdívku quick and dirty, budete tomu rozumět :-). A budete si moc vzít další Issue

MOC A MOC DEKUJI

matej-sveda commented 2 years ago

@ZdenekMen trapi me tady porad ta posledni vec, ze pri kazdem vytvoreni nabidky (tj. registraci), je potreba se verifikovat pres email, aby se nabudka vytvorila... z logiky veci by nemelo byt nutne se verifikovat vicekrat, ale je to tak postaveno v BE a me se tu verifikaci nedari obejit... podle vseho totiz s kazdou nabidkou vznika i dalsi uzivatel - ma nove voulnteerId, i kdyz ma stejny email a ostatni detaily... jeste tomu dam dnesni vecer, ale kdyby se to nepodarilo... je schudne aby to fungovalo tako? tj. bude mozne pridavat nabidky, jak jsi navrhl. ale pokazde je nutne klikat na verifikacni link z emailu (az pak se totiz nabidky fakticky vytvari) image

matej-sveda commented 2 years ago

...delam to v jine branchi nez @martinsik , tak se nelekej te prazdne prave casti :) pokud vim tak on uz ma editaci profilu temer hotovou, ne-li uplne

ZdenekMen commented 2 years ago

Ahoj Mateji @matej-sveda , hmmm, to je tedy boj . Tak to je na tvurcich systemu. @honzasladek Honzo, prosim te, muzes nam poradit s tou Matejovou otazkou vyse ? DEKUJI VSEM

matej-sveda commented 2 years ago

@honzasladek vecer mam v planu zkopirovat kod, ktery verifikuje uzivatele na api/verify.ts a pouzit ho v tom submitu registracniho formulare, ale i kdyby to fungovalo, neni to moc elegantni reseni

honzasladek commented 2 years ago

@matej-sveda To určitě nedělej. :) Stačí založit novou nabídku, ne? K uživateli to jen navázat. (Ale uživatel už bude ověřen a jen k němu přibude nová nabídka. Ani žádný mail není potřeba.)

Odpoledne pošlu cca návod, jak bych na to šel.

matej-sveda commented 2 years ago

@honzasladek to budu rad, dekuju, ja totiz neprisel na jiny zpusob vytvoreni nabidky nez prostrednictvim registrace noveho uzivatele

honzasladek commented 2 years ago

@matej-sveda Hmm, koukám, že to bude trošku programování, protože teď je to fakt hodně spojené. Každopádně je potřeba:

1) Povolit v Contember ACL operaci create nad Offer. Zhruba takhle: create: allField(model, 'Offer', 'self'), nad řádek 425 souboru acl.ts. (Pak je potřeba ještě vygenerovat migrace.)

2) Tím můžu s API klíčem Volunteera zakládat nové nabídky a tedy v GraphQL API mi přibude endpoint něco jako createOffer mutation. Do něj pak půjde poslat ta část z register.ts createInput, kde se řeší Offers. Tj. založit novou offer a connectnout jí na uživatele, za kterého jsem přihlášený. V moje-nabidky.tsx je const volunteerId, ve které je ID uživatele, za kterého jsem přihlášen a na kterého to chci connectnout.

No a that's it. Možná by bylo fajn trošku zrefaktorovat to register.ts a vyjmout tu část zakládání offers bokem, abych jí mohl volat jak tady, tak při zakládání nové nabídky z uživatelského profilu. Ale to jenom, aby se zbytečně neduplikoval kód, to vytvoření nabídky je trošku složité tím, jak všechny formuláře jsou dynamické a můžou si je lidi z Konsorcia na backendu měnit.

matej-sveda commented 2 years ago

@honzasladek vecer se dam do toho, ale nebude problem ze nerozjedu lokalne contenber? nebudu to moct otestovat dokud to nebude v main branchi

honzasladek commented 2 years ago

@matej-sveda Jako není to zrovna ideální. Ale tak můžu ti udělat tu úpravu s tím ACL a ty to pak budeš vyvíjet proti betě?

matej-sveda commented 2 years ago

@honzasladek ano prosim

honzasladek commented 2 years ago

@matej-sveda Máš to tam.

ZdenekMen commented 2 years ago

Ahoj Mateji @matej-sveda , tato funkčnost nějak proklouzla na Produkci včetně barevných stavů nabídek od Vila @ViliamV (objevil jsem to včera náhodou při testování ukrajinštiny). Hned jsme to vyzkoušel - funguje to perfektně s jedním problémem.

Takto založená nabídka nebyla VČERA jinde (FE Nabídky pomoci) vidět. DNES ráno se tam ale objevila. Tuším, že to bylo chybějícím potvrzením registrace z mailu, což Honza @honzasladek z BE snad přes noc "napravuje".

Prosím tě nešlo by to zobrazovat hned tj asi nahodit stav, že je nabídka plně OK?

MOC VŠEM DĚKUJI

Dnes jsem okolo 14.00 zadal 2 případy Doprava TestX a TestY, na BE jsou vidět, na FE ne.

matej-sveda commented 2 years ago

ahoj @ZdenekMen , jen abych to spravne pochopil.. rikas, ze jsi pridal nabidu pod existujicim uzivatelem (ze stranky moje-nabidky)...a na FE se objevila az dalsi den? s verifikaci pres email by to nemelo souviset, protoze takovy uzivatel uz je verifikovan.... po te co jsi ji tam pridal, nebyla videt ani mezi tema tvyma nabidkama po refreshi stranky?

honzasladek commented 2 years ago

Já souhlasím s @matej-sveda, měly by být vidět hned. Pokud nejsou, tak je spíš někde něco divně u zobrazení.

Jedna z věcí co mě napadá (ale může být blbost, nekoukal jsem do kódu) nekontroluje se na FE, zda jsou nabídky přeloženy? Protože to nejsou hned, ale s nějakým zpožděním (+- 1 hodina).

ZdenekMen commented 2 years ago

Ahoj pánové @honzasladek @matej-sveda , díky moc, tak ty dnešní Nabídky TestX a testY už vídím, zadal jsme TestZ a ten nevidím (restartoval jsme celý chrome, takže refreshem to není) Posoudím to překládání a vidím, že se fakt mezitím přeložily

ZdenekMen commented 2 years ago

Ahoj pánové @honzasladek @matej-sveda, omlouvám se, Honza měl pravdu, je to fakt tím překladem. není nad to dělat testy, které v normálním životě nejsou moc reálné ;-( děkuji za nedělní podporu