poff-bnff / web2021

1 stars 0 forks source link

Vorm kasutajale #534

Open jaanleppik opened 2 years ago

jaanleppik commented 2 years ago
jaanleppik commented 2 years ago

Profiili loomise ja toimetamise õigus peaks tekkime kõigile Industry lehele sisse loginud kasutajatele St kõigil PÖFFi kasutajatel peab tekkima õigus ka industry.poff.ee (ja discoverycampus.poff.ee) lehele sisse logida

mpennaste commented 2 years ago
  • [ ] Luua vormi PUG

Vb sellise lihtsalt PUGis vormi tegemise võiks just nt anda kellelegi katsetada esimeseks asjaks, et toda PUGi jms harjutada sutsu?

jaanleppik commented 2 years ago
  • [ ] Luua vormi PUG

Vb sellise lihtsalt PUGis vormi tegemise võiks just nt anda kellelegi katsetada esimeseks asjaks, et toda PUGi jms harjutada sutsu?

See võtaks liiga palju aega... Ma tahaks ASAP ise modima hakata seda vormi, nii et vormi starterit võimalikult lihtsal viisil tahaks esimesel võimalusel.

mpennaste commented 2 years ago

Okei, kas toimetad selle kallal ise või midagi teha vaja enne?

jaanleppik commented 2 years ago

Okei, kas toimetad selle kallal ise või midagi teha vaja enne?

mina hakkaks toimetama siis, kui PUG on olemas ja selle kaudu saab valitud collectionisse (näiteks PERSON alguses) infot kirjutada ja collectonist infot kuvada. E enne tahaks PUG'i, mis minu frondi kasutajale nähtav (industry lehel Eventivaliga autentides näiteks, või ka PÖFFi lehel autentides) ja mis lubab ühte kirjet PERSON collectionis luua, lugeda ja uuendada

mpennaste commented 2 years ago

Sellist universaalset nüüd luua oleks üpris võimatu (arvestades, et relatsioonid jne jne), pigem ikkagi teha alguses valmis PUG vorm, et teada täpselt mis andmeid sealt soov kirjutada ja ikka kindla collectioni tarbeks. St enne ikka konkreetne vorm ja siis funktsionaalsus sinna järele (nagu meil hetkel nt signup või userprofile).

jaanleppik commented 2 years ago

Sellist universaalset nüüd luua oleks üpris võimatu (arvestades, et relatsioonid jne jne), pigem ikkagi teha alguses valmis PUG vorm, et teada täpselt mis andmeid sealt soov kirjutada ja ikka kindla collectioni tarbeks. St enne ikka konkreetne vorm ja siis funktsionaalsus sinna järele (nagu meil hetkel nt signup või userprofile).

Ma mõtlengi konkreetset, alguses person collection ja mõned väljad, eesnimi, perenimi ja veel mõni lihtne väli. Siis hakkaks välju lisama ja ehitama korraga collectionit kui ka vormi, ja samal ajal saab tegeleda õiguste arendamisega jms

mpennaste commented 2 years ago

Sa mõtled siis ikka seda PersonPros või mida? Eile käis ju jutt sellest või saan ma väga valesti aru.

jaanleppik commented 2 years ago

Alustaks Personiga. PersonPro on testiks, prototüübiks jne, mille alusel teen PERSON'it ymber siis

mpennaste commented 2 years ago

Et siis praegu esimesel võimalusel oleks vaja lehte kus iga inimene võib lehel lisada persoone?

jaanleppik commented 2 years ago

Et siis praegu esimesel võimalusel oleks vaja lehte kus iga inimene võib lehel lisada persoone?

jah, toimetada kindlat persooni kirjet ja lisada uusi persooni kirjeid

jaanleppik commented 2 years ago

Dropdownkide täitmine dünaamiliseks: Dropdown'ide sisu on Strapis collectionitena, kolmekeelsete väärtustega collectionitena. Tahaks dropdownide sisu täita nende collectionite sisuga, mis on olemas lehe buildimise hetkel. Collectionite / dropdownide sisu võiks olla YAML'is. kas siis eraldi loodud YAML'is või includetud collectioni infot sisaldavas YAMLäis - st iga sellise collectioni sisu sisaldava dropdownli lisamisel tube includeda ka vastava collectioni YAML

jaanleppik commented 2 years ago

Dünaamiline vormi ridade lisamine: vormi kaudu on vaja mitu korda sama tyypi sisu lisada: näiteks mitu pilti, mitu filmi, jne Kasutaja peaks saama luua uue tyhja rea / elemendi Jaan korratava galerii pildi vormi elemendi, selle võiks siis korratavaks muuta.

jaanleppik commented 2 years ago

Piltide lisamine vormi kaudu:

  1. proffi profiilipit / näopilt: ruudukujuline, ainult 1 pilt
  1. proffi galeriipilt, mitu tk - korratava vormi komponendi abil.
jaanleppik commented 2 years ago

vormis on fieldset, kus on profile_img faili upload ja photographer text väli ja img_year numbriväli. Need kaks viimast peaks siis kirjutama Caption väljale, eraldajaks ", " (E neile pole siis personi collectionis eraldi muutujaid loodud.)

jaanleppik commented 2 years ago

pyydsin isada pildi valideerimise ja saatmise datasse lisamise koodi osa, nagu userProfile javascriptis on - aga vorm nyyd ei tööta. (edited) võtan muutuse tagasi ja seda koodi lugedes tekkis veel küsimus, et kuidas sama koodi panna MITME pildi saatmisel tööle? nii et vajan abi selle pildi javascriptiga saatmisel

jaanleppik commented 2 years ago

Profiilipildi üleslaadimine industry lehel live vormil minul testides ei toimi. @mpennaste palun vaata üle.

mpennaste commented 2 years ago

Mis sa täpselt mõtled selle all? Profiilipilti või personFormi pilti? Profiilipilti Industrys ei kasuta ning personFormi kohta eile kirjutasin Slackis.

mpennaste commented 2 years ago

Ühe pildi sain lisama. image Alt ja caption lisatakse ka (nimi, originaalfailinimi - hetkel). Lisaks veel failinimi prefixiga "PF_" ja seal järel slugifyst läbi käinud ees- ja perenimi. image

Kui pull request merge'ida ja maini tõmmata (Strapi restarteerub), siis tuleb Authenticated kasutajale anda personForm endpointi õigus. https://admin.poff.ee/admin/settings/users-permissions/roles/1 (Lehe allosas USERS-PERMISSIONS->USER->personForm)

Pull request: https://github.com/poff-bnff/web2021/pull/540

jaanleppik commented 2 years ago

Tegin ymber aadressi collectioni ja komponendi info struktuuri, ja kõigepealt muutsin collectioni nime PopulatedPlace -> Municipality Aadressi collectioni ja komponendi sees muutsin ka sellele collectionile viitavad relatsioonid ära.

jaanleppik commented 2 years ago

Ja lisasin aadressi collectionisse ja komponenti text välja popul_place eestis on see alev, alevik, küla, linnaosa, linn (osad linnad st, Tallinn ja suuremad linnad on municipality collectionis) esialgu on see lihtsalt tekstiväli. tegelikult on ka see nimekiri eestis lõplik, nii et saab teha collectioniks idee poolest.

Palun datemodeli update'i

jaanleppik commented 2 years ago

@LiisKasper palun kriitilist pilku aadressi väljade muutujate nimetamisele vormis: järjekindel oleks naad kõik läbivalt addr prefiksiga teha, aga edasi on strpis muutujate niede syntaks juba erinev. alakriipsudega vs CamelCase. praegu siis natuke ebajärjekindel seis. ja kas siin kah on muutuja nime pikkus probleemiks? seda siis ainult info saatmise sammul Strapis midagi pikemaks ei teinud

LiisKasper commented 2 years ago

Strapis tunduvad nimetused hetkel hästi. Kõik läbivalt alakriipsuga ja nimetajate pikkus hea. Nimetajate pikkus on probleemiks siis, kui strapis luua kahe objekti vahel relatsioon. Relatsiooni tabeli nimetuse keevitab strapi kahest poolest kokku, pika nimetuse puhul lõikab Strapi tabeli nime lõpu ära. Kui tekib teine nimetus, mis juba olemasolevaga sarnane, siis meil on tekkinud olukordi, kus kahe kokku lõigatus tabeli nimetused on täpselt samad, siin läheb Strapi katki. Vormis nimetamine otseselt ei mõjuta seda, kuidas infot Strapisse saadetakse. Hetkel tuleb erinevus sellest, et oled lisanud kõigile nimetajatele ette addr. Siin saad ise valida, kas lisad need alakriipsu v CamelCase loogika alusel. Kui lisad, siis teeksin seda läbivale sarnaselt, alakriipsuga. Mina ei lisaks addr_ eesliidet. Scriptis on need nimetajad ühes objektis address sees ja vormis saaks lisada fieldset kõrvale lihtsalt kommentaari, et koodi kirjutajale/lugejale selgem oleks. Siis oleks nimetamine sama Strapiga. @mpennaste vist palus, et nimetajad oleks vormis samad, mis Strapis?

jaanleppik commented 2 years ago

prefiksit on vaja näiteks country juures - sest küsime country't ka väljaspool aadressi. ja img juures - kui strapis on korratav img media komponent - siis on vormis vaja ikka eraödi fotod nimetada, kui neile ka caption välja vaja täita.

Saan ümber teha küll enamus, aga siis oleks hea ka nende muutujate mapping tabelit näha?

LiisKasper commented 2 years ago

Kui kuskil vaja, siis pigem juba läbivalt, arvan mina.

jaanleppik commented 2 years ago

@mpennaste nimetame persooni

jaanleppik commented 2 years ago

personForm'is fieldset#actorinfoset selle näitamine peaks sõltuma valitud RoleAtFIlm (vormis Professional role) väärtusest. Tahaks teha listi väärtustest, millede puhul fieldset#actorinfoset muutub nähtavaks. Actor, Dubbing actor, Operetta singer jne. E kuhugi tahaks luua whitelisti väärtustest.

jaanleppik commented 2 years ago

personForm'is other_lang on praegu tehtud multiple select väljana. Kas neid saab probleemideta Strapisse kirjutada? teine variant on teha korratava template'na, nagu gallery image oleme teinud.

jaanleppik commented 2 years ago

Lisasin kaks vormi sektsiooni, mis peaks kirjutama Filmography collectionisse. profEducation section ühele osale väljadest Filnography collectionis worksFilmography teisele osale väljadest Filnography collectionis, sh osad väljad kattuvad.

@mpennaste

jaanleppik commented 2 years ago

Kuidas teha SAVE PROFILE nupu aktiveerimine väljade sisu muutuse peale? (nagu Strapis) kui laen profiili andmed, siis SAVE nupp on inaktiivne kuni esimese muutuseni.

jaanleppik commented 2 years ago

addRoleAtFilm() funktsiooni nimetasin kah, kuna arvan ,et peame multiselecti tegema ikka templatedega - sest pikas dropdownis valitud väärtusi ei saa kasutajale hästi täidetud vormis näidata… ja tegin template alguse kah ära, mitme pildi näitel

jaanleppik commented 2 years ago

lisasin audio faili üleslaadimise vormile

.form_group
                        label.person_profile_label(for='audioreel')= self.label.userprofile.audioreel
                        input.login_input#audioreel.form_control(type='file' name='filename' accept="audio/*" onchange ='validateImageAndPreview(this.files[0], "profileImage", "profile")')

funktsioonide ja muutujate nimed on kindlasti poolikud, label ja input'i ID on ainukestena saanud väärtuse 'audioreel'

See oleks ainut 1 audiofaili üleslaadimise funnktsioon.

@mpennaste palun tee JS osa sellele

jaanleppik commented 2 years ago

eesnimi + perenimi e person collectioni väli firstNameLastName kas täita saa Strapis või vormi JS koodiga? @LiisKasper pallun mõtle, kuidas sinu meelest oleks parem? mis on lihtsam meile ja mis on lihtsam Strapile?

profiilipilt läheb praegu valele väljale. Meil on personi all picture väli varasemast ja vormi jaoks loodud profile_img väli. praegu kirjutab vorm picture väljale. aga tegelikult hakkan jõudma arvamusele, et nii ongi hea :) pole vaja muuta. kustutan liigse profile_img muutuja ära.

mpennaste commented 2 years ago

Kui seda vaja, siis pigem jätta Strapile.

Profiilipildi jah panin sarnaselt profiilile ja testimiseks algul sinna (profile_img alles hiljuti avastasin, et olemas), reaalselt väga vahet pole, oleks suht lihtne muuta, aga kui nii on OK, siis... las olla :)

jaanleppik commented 2 years ago

@LiisKasper kas teed selle nimede kokku liitmise Strapis?

LiisKasper commented 2 years ago

@LiisKasper kas teed selle nimede kokku liitmise Strapis?

Lisasin PUT endpointi Strapis nimede kokku liitmise, firstNameLastName välja täidetakse sealt.

jaanleppik commented 2 years ago

Pildi caprion välja täitmine: praergu on nii alt text väljal kui caption väljal failinimi, aga peaks olema photographer ja img_year väljade sisu, komaga eraldatuna. Kui @mpennaste annab @LiisKasper'ile otsad kätte - kas Liis saaks selle Strapis captin välja kirjutamise teha?

jaanleppik commented 2 years ago

Kas saab nii, et Profession väli, mis on korratav, aga ka kohustslik, oleks vormi avades koha nö avatud ja nähtav, et selle template'i esimene default olek oleks nähtav? Kui teeb professioni template'i by default avatauks siis mitme värrtuse saatmine ei toimi - saadetakse ainult 1 väärtus, mis on sisestatud nupust avatud teisele väljale. e by default avatud välja kaudu info edastamine praegu ei toimi

jaanleppik commented 2 years ago

profession väli (korratav) ja ka education ja filmography - võiks olla ka kustutatavad, nagu gallery image korratavad komponendid @mpennaste, kas saab lisada samasuguse kustutamise nupu ja edufilmography juurde, nagu galeriipiltide juures on?

jaanleppik commented 2 years ago

tundub, et audioreel'i uploadil on image controll peal - saan profiilipildi juures veateate, et file is not an image, kui püüan salvestada vormi koos mp3 failiga audioreeliks

mpennaste commented 2 years ago

Pildi caprion välja täitmine: praergu on nii alt text väljal kui caption väljal failinimi, aga peaks olema photographer ja img_year väljade sisu, komaga eraldatuna. Kui @mpennaste annab @LiisKasper'ile otsad kätte - kas Liis saaks selle Strapis captin välja kirjutamise teha?

Sellega on keeruline, sest uurisin ja puurisin, aga front poolelt seda Strapile koos pildiga ei õnnestunud anda. Üks variant oleks anda eraldi objekt ja siis mappida pildi mingi võimalikult unikaalse info järgi see. Ala pildi failinime järgi, nt kui pilt nimega "299845826_5945105425503001_4802689012020278901_n.jpg" Ja siis backis juba miski magic poogib pildile juurde selle kui on ala objekt:

[{ "299845826_5945105425503001_4802689012020278901_n.jpg": { "alt": 'Toivo', "caption": '2022' } }]

Siin võib muidugi juhtuda, et inimene võtab ühest kataloogist oma arvutist ühe faili nimega "pilt.jpg" ja teise teisest kataloogist sama nimega "pilt.jpg" ja siis ei funka nii nagu võiks.

jaanleppik commented 2 years ago

kui strapi tasemel media library objekti sisse kirjutamine toimib - kas siis ei oleks varint: teha korratav pildiallkirja komponent, mille osadeks on tekstiväjad, mida vormis pildi juurde küsime (fotograaf ja aasta ja järgmiseks ka tavamõistes pildiallkiri e kirjeldus) ja mingil viisil sidumine pildiga, viide pildile ja siis Strapis käivitada kood, mis peale vormi kaudu info salvetamist liidab pildi media library objektile nende 2-3 tekstivälja sisud, Caption väljale. E e esimene samm on koguda Vormist info tekstiväljadele, mis on küll komponendis - mida see keerulisemaks muuda? Ja teine samm on strapi tasemel liita tekstiväljade sisu Media library objekti Caption väljale..

VÕi mõni teine viis, kuidas saame kätte ja salvestame Strapis ära pildi, selle juurde kuuluvate tekstiväljade sisu koos seosega pildile; ja teise sammuna kirjutame Media Library objekti selle info kokku.

mpennaste commented 2 years ago

See teeks vist sama välja suhteliselt, et ikkagi liidame Strapis hiljem pildile külge.

jaanleppik commented 2 years ago

teiste keelte valik, lisaks native language'le: esimest korda vormi täites saab sellega hakkama, et valid controlit / commandi all hoides oma keeled ära. Korraga ylevaadet sellest kasutajal silme ees pole, aga saab hakkama. Aga kuidas kasutajale näidata tema valitud keeli siis, kui täidame vormi strapis oleva infoga? Nii et ses mõttes oleks teiste keelte valik selline, nagu professionite valik - e yhe väljaga korratava komponendina.

mpennaste commented 2 years ago

Sama küsimus tekib hiljem nende korratavate asjadega, ses osas kui data hakkab vormi Strapist tulema, siis läheb nendega üpris keerukaks ja huvitavaks.

jaanleppik commented 2 years ago

Proovisin vormi kaudu persooni lisada. Täitsin ära maiken Piusi ankeedi: kõik pildid, kogu olemasolev ankeedi info ja, 2 education osa ja 1 filmography osa. Salvestasin Pildid jõudsid librarysse, aga persooni ei tekkinud.

jaanleppik commented 2 years ago

tundub, et teise salvestamise peale loodi persoon, aga siis enam ankeedis pilte ei olnud, st need olid ankeedist kustutatud, kuna need said saadetud Strapisse ilmselt. Ja ka education ja filmograafia puudu. Hakkan algusest uuesti

jaanleppik commented 2 years ago

Ei esimesel salvestamisel jüuavad Sterapisse ainutl pildid. Persoon ei jõua Filmograafia collectionisse filmograafia asjad kah ei jõua. Ankeedit kaovad peale esimest salvestamist pildid ja filmofraafia info, muu info säilib ankeedis.

jaanleppik commented 2 years ago

proovinJaan A Leppik'ut luua pildid OK - 2 gallery pilti Profession e RoleAtFilms ei jõua kohale kolmest other languagest jõuab ainult 1 date of birth ei jõua, kuna ma ei valinud päeva ja kuud - ainult aasta muutsin ära

jaanleppik commented 2 years ago

role at films - ainult teine tuleb Strapisse, esimene ei tule e see, mis on juba vaikimisi avatud - ei tule other languages - tuleb ainult esimene