poff-bnff / web2021

1 stars 0 forks source link

CourseEvent collection ja PUG #498

Open jaanleppik opened 2 years ago

jaanleppik commented 2 years ago

Eesmärk on kirjeldada kursust collectioniga CourseEvent ja luua sellele PUG https://admin.poff.ee/admin/plugins/content-type-builder/content-types/application::course-event.course-event

PUG i aluseks võiks võtta olemasoleva Industry_Event collectionile vastava PUG'i industry_event_index_template.pug Tegin sellest koopia, nimega course_event_index_template.pug ja kopeerisin sinna industry_eventi pug'i sisu

Töötabel info struktuuride võrdlemiseks https://docs.google.com/spreadsheets/d/1U-SaJ47y7Ja5h0dARFvmsWg8HsApcFNmjHmHkjUOym4/edit?usp=sharing

täiendan / muudan course_event_index_template.pug faili

LiisKasper commented 2 years ago

Tähelepanek et CourseEvent sees väli preconditions viitab collectionile CourseEvent e iseendale

J: mõte on märkida kursusele registreerumiseks / kursusel osalemiseks eeltingimuseks olevaid läbitud kursusi. näiteks on kursusel K osalemise eelduseks

See pole hetkel kaugeltki esimene prioriteet, aga idee on selline.

jaanleppik commented 2 years ago

Palun võimalust seda CourseEventi buildida

LiisKasper commented 2 years ago

@mpennaste kuna tegemist on uue mudeliga, kas nimetused võiksid läbivalt olla _ (alakriipsuga), mitte varasema põhjal!? (videoUrl, isLiveEvent, isPublicEvent)

LiisKasper commented 2 years ago

@jaanleppik miks duration_time mitte duration? selgem, keeleliselt õigem? Ok, näen, et meil on IndustryEvent all ka durationTime. Andmebaasis on ka duration, see tekitas segaduse. Aga nimetamise osas, kumb jääb?

LiisKasper commented 2 years ago

https://pff-dev.slack.com/archives/G017XBQ5UJY/p1647431021560829

[1:43 PM](https://pff-dev.slack.com/archives/G017XBQ5UJY/p1647431021560829)
[Martin Pennaste](https://app.slack.com/team/U018F1KKDFA) Kõik keeleasjad jms ala title_et, name_et peaks kindlasti alakriipsuga olema, et rueten aru saaks asjadest jne, domeen ka siis domain/domains jms.
Aga no kuna me eile siin arvasime kõik, et pigem teeme täitsa uue asja, et siis vana ka funkaks seni, siis ülejäänud nimetusi võib muuta vist, peaasi, et kusagile kirja saab kui täitsa erinevaks muudetud sama välja nimi, et hiljem hea võtta oleks. Mulle meeldib alakriipsuga väga hästi :)
LiisKasper commented 2 years ago

Muudan siis eeltoodud kolme nimetuse kirjapilti (alakriipsudega vastu)

mpennaste commented 2 years ago

@mpennaste kuna tegemist on uue mudeliga, kas nimetused võiksid läbivalt olla _ (alakriipsuga), mitte varasema põhjal!?

Kuna tegu uue mudeliga jms, siis läbivalt kõik alakriipsuga kõlab väga hea plaan.

LiisKasper commented 2 years ago

Kas IndusrtyEvendist andemete ümber tõstmisel, keelte tõlkeväljad jäävad tühjaks? St nt title_et jääb tühjas või paneme sinna inglise keelse sisu, sama title_ru, slug_et, pulblish_et jne

jaanleppik commented 2 years ago

paneme ingliskeelse sisu eesti ja vene keele alla

jaanleppik commented 2 years ago
title_private Text
title_et Text
title_en Text
title_ru Text
slug_et UID
slug_en UID
slug_ru UID
short_descr_et Text
short_descr_en Text
short_descr_ru Text
description_et Rich text
description_en Rich text
description_ru Rich text

start_time | Datetime

duration_time | Time

end_time | Datetime

location | Relation with Location

course_module | Component (repeatable) title_private | Text title_et | Text title_en | Text title_ru | Text contents_et | Rich text contents_en | Rich text contents_ru | Rich text images | Media

videoUrl | Text

order | Number

from | Datetime until | Datetime

public | Boolean

publish_et | Boolean publish_en | Boolean publish_ru | Boolean

credentials | Component (repeatable)

rolePerson | Component (repeatable) order | Number role_at_film | Relation with RoleAtFilm person | Relation with Person actor_role | Text

roleCompany | Component (repeatable) order | Number roles_at_film | Relation with RoleAtFilm organisation | Relation with Organisation

domains | Relation with Domain

event_mode | Relation with EventMode

trailer | Component (repeatable) url | Text source | Enumeration

industry_categories | Relation with Industry_Category

isLiveEvent | Boolean

isPublicEvent | Boolean

public | Boolean

preconditions | Relation with CourseEvent

moodle_id | UID

product_categories | Relation with Product_Category

publish_et | publish_en |   publish_ru |  

posters | Media images | Media

videoUrl | Media

LiisKasper commented 2 years ago

Andmete transpordiks lisan CourseEvent collectionile välja remote_id. Hiljem seda vaja pole ja kustutan.

LiisKasper commented 2 years ago

Üritan kustutada ühte CourseEvent kannet ( id 4 https://admin.poff.ee/admin/plugins/content-manager/collectionType/application::course-event.course-event/4). Saan errori (mitte logides). Oskab keegi aimata, kuidas seda paranada v kuidas see tekkinud on? Screenshot from 2022-03-24 10-51-19

jaanleppik commented 2 years ago

Ei oska aimata. Kusagil on viide Userile, ... mida pole modelis kirjeldatud? kui püüan eesti keelde tõlkida. Aga your FILTERS contain - mida see filters siin tähendab?

LiisKasper commented 2 years ago

Ei tea, aga hetkel ei saa ühtegi kannet kustutada. St see mudel on kuidagi katki, Aga mina ei oska aimata, kust see tuleb, pole enne näinud. Kas kellegi teisel õnnestub kustutamine?

mpennaste commented 2 years ago

Üritan kustutada ühte CourseEvent kannet ( id 4 https://admin.poff.ee/admin/plugins/content-manager/collectionType/application::course-event.course-event/4). Saan errori (mitte logides). Oskab keegi aimata, kuidas seda paranada v kuidas see tekkinud on? Screenshot from 2022-03-24 10-51-19

Mina tean, pelgalt seetõttu, et kunagi pool päeva või terve päeva vms vaagisin, et mis jama on ja siis kontoris ka mingi hetk seda vaidlesime. See on see teema mis kunagi Siim tegi, et me teaks, mis kasutaja millise asja ära kustutas (muidu läheb see kirja kes eelnevalt muutis). Ja lisaks vist ka selleks, et kustutades samuti build jms jooksma läheks (kui nii määratud). Tee copy-paste random mudelist millel pole mingit eriloogikat ega ehitusloogikat veel küljes, nt Hall'i alt (veidi parem, seal build-loogika ka) või veel lihtsam... https://github.com/poff-bnff/web2021/blob/main/strapi/strapi-development/api/user-profiles/models/user-profiles.js Viimases siis oluline rida 24 kus parameeter "user" siis eemaldatakse enne kui Strapi asja kustutada üritab, sest Strapile ei meeldi kui talle mingit üleliigset kraami kustutamisel anda. Tahab ilmselt vaid ID-d mitte nii ID-d kui random parameetrit "user".

Ma ei tea kas sai selgem, aga ühesõnaga uue mudeli loomisel tuleb juurde pookida, siis saab edukalt kustutada :)

LiisKasper commented 2 years ago

Aitäh! Poleks selle peale kohe tulnudki, aga seletab :)

LiisKasper commented 2 years ago

@jaanleppik industryEvent ümber kirjutamisel courseEvents collectionisse. Saan error Need kaks sinu loodud üritust on ka industryEvents all olemas. Üks identse slugiga, mida ei saa olla: https://admin.poff.ee/admin/plugins/content-manager/collectionType/application::course-event.course-event/3 https://admin.poff.ee/admin/plugins/content-manager/collectionType/application::industry-event.industry-event/191 ja sarnase slugiga, milles kirjutaks teise courseEvent juurde https://admin.poff.ee/admin/plugins/content-manager/collectionType/application::course-event.course-event/2 https://admin.poff.ee/admin/plugins/content-manager/collectionType/application::industry-event.industry-event/102

Kas jätan antud industryEvenedid kirjutamisel vahele? Kui mõlemad peaksid säilima, siis esimese puhul on vaja ühel v teisel muuta nime.

jaanleppik commented 2 years ago

muutsin neil kahel ise tehtud testikal nimed ja slug'id ära. et las jäävad alles. ainult testimise / arendamise ajaks. pärast siis kustutan, nimes kah TEST sees

LiisKasper commented 2 years ago

IndustryEvents andmed on tõstetud CourseEvents sisse. Kursuseid on kokku 6. Lihtsam on need käsitsi ümber tõsta, kui päringuid kirjutada. St tõstsin käsitsi @jaanleppik esimene error kohe test courseEvent viitam moodle_id 2 (https://admin.poff.ee/admin/plugins/content-manager/collectionType/application::course-event.course-event/2) Sama tahaks teha ka (https://admin.poff.ee/admin/plugins/content-manager/collectionType/application::course.course/6) hetkel jäi test courseEvent alla alles ja teisele lisamata. Lisaks päris mitmed väljad, mis ei sobitu kokku v tahaks oma kirjeid (nt location saaks olla online aga ilmselt mitte veebikino ONLINE jne) Palun vaata üle courseEvents alates id 1027 kuni 1032, tõstsin üle nii palju kui uude mudelisse sobitus, aga päris isetegevust tegema ei hakanud, va publish_et jne

Kui kõik tunub nii nagu tahetud, siis võib andemete liigutamise collectionitest IndustryEvents ja Courses collectionisse CourseEvents lõpetatuks lugeda.

mpennaste commented 2 years ago

@LiisKasper / @jaanleppik Kas kusagil harus on datamodelis ka see CourseEvents olemas?

LiisKasper commented 2 years ago

Ei, veel mitte. Nüüd on. Datamodel harus. Maini pole mõtet peale tirida vist veel, ainult lisakoormus fetcheritele

jaanleppik commented 2 years ago

Siin võib kyll dration_time panna mu meelest, sest see on uus collection ja PUG - mõjud teada.

mpennaste commented 2 years ago

See suur, avastasin et mul koodi jaoks asju puudu Industry osas: project_type channel industry_projects publish_from/-until

jaanleppik commented 2 years ago

Need esimesed 3 jätsin teadlikult välja. Aga kaaludes lisan ikka relatsiooni industry projektidele kah. Publish from / until lisan. Neid pole tõesti põhjust välja jätta.

jaanleppik commented 2 years ago

Üldine plaan selle collectioniga: See peaks asendama nii senist Course collectionit kui ka senist IndustryEvent collectionit. E nii aastaringsed kursused kui ka industry ajal toimuvad paneelid / töötoad oleksid samamoodi kirjeldatud.

Mõtlen, et kas ka tavalised sündmused - peod, vastuvõtud, kinnised üritused ja muu, mida tahame kodulehel näidata, kas avalikukt või vaid sisse loginud kasutajatele - võiks olla selles collectionis?

mpennaste commented 2 years ago

Need esimesed 3 jätsin teadlikult välja.

Aga... project_type ju meil isegi filtrites ja puha? (link)

mpennaste commented 2 years ago

Puudu filmikoolil: tags titleOriginal presentedBy events

mpennaste commented 2 years ago

Kas on mingi mõjuv põhjus, miks courseEvent all credentials on korratav? Ma oma mõtteis näen, et loogilisem oleks pigem nagu on course all lahendatud - st, et ei oleks korratav. image

jaanleppik commented 2 years ago

Ei ole mõjuvat põhjust, näpukas on. Parandasin.

jaanleppik commented 2 years ago

Need esimesed 3 jätsin teadlikult välja.

Aga... project_type ju meil isegi filtrites ja puha? (link)

Seda minu meelest kasutati kas vähe või valesti - e industry inimesed ei saanud sei kah enam aru, mis see seal tähendas, industry eventi juures. nii et parem jätame ära

mpennaste commented 2 years ago

Super, tänud! Kustutan selle project_type siis koodist ka ära :)

mpennaste commented 2 years ago

Filmikooli osas on ka keeled (languages) puudu. Ja duration time tundub olevat täitmata.

jaanleppik commented 2 years ago

Duration time jääbki paljudel juhtudel täitmata. Näiteks kui on kuuajaline kursus, siis on alguse kuupäeva ja lõpu kuupäev, aga tundides väljendatud duration time siis ei panekski.

jaanleppik commented 2 years ago

Lisasin Languages relatsiooni kah Course Eventile

mpennaste commented 2 years ago

Live veel ei paneks, aga ehitada ja localis testida peaks saama. Haru CourseEvents. Muudatusi saab vajadusel näha: https://github.com/poff-bnff/web2021/compare/main...CourseEvents

Andmeid tuunib: helpers/fetch_courseevents_from_yaml.js

@jaanleppik tehtud index template nimetasin ümber (industry omaks). St üldse tegin kokku kolm tk - igale lehele oma, et saaks eraldi tuunida (Industry ja Filmikool kohati erinevad siiski, ühel seoseid/asju mida teisel pole jne). _industry_courseevent_index_template.pug filmikool_courseevent_index_template.pug discamp_courseevent_indextemplate.pug (jäi ebaselgeks, et kas Discamp ka CourseEvent peale)

Source kataloogis on kataloogid _coursescourseevents (Filmikool, veebis /courses-courseevents) ja _industry_courseeventssearch (Industry[/Discamp], veebis /industry-courseevents-search) mis siis on listivaateks.

Enne live panekut teha:

... ja kui miskit puudu vms, siis õhtul saan ka vaadata.

jaanleppik commented 2 years ago

Localis, branchiga CourseEvents, ehitasin Discampi lehe, aga nii localhost:4000/courses-courseevents localhost:4000/en/courses-courseevents localhost:4000/kursused-courseevents kui ka localhost:4000/industry-courseevents-search annavad 404

jaanleppik commented 2 years ago

industry local build samas branchis annab

ERROR:
/Users/jaanleppik/GitHub/web2021/ssg/source/industry_mycal/index.pug
Cannot read property 'length' of undefined
mpennaste commented 2 years ago

1.

Localis, branchiga CourseEvents, ehitasin Discampi lehe, aga nii localhost:4000/courses-courseevents localhost:4000/en/courses-courseevents localhost:4000/kursused-courseevents kui ka localhost:4000/industry-courseevents-search annavad 404

See:

Source kataloogis on kataloogid _coursescourseevents (Filmikool, veebis /courses-courseevents) ja _industry_courseeventssearch (Industry[/Discamp], veebis /industry-courseevents-search) mis siis on listivaateks.

2.

industry local build samas branchis annab

ERROR:
/Users/jaanleppik/GitHub/web2021/ssg/source/industry_mycal/index.pug
Cannot read property 'length' of undefined

Tõsi, minu paha, viimasena vabanesin mujal koodis types filtrist ning sealt olin täitsa unustanud eemaldada. Nüüd kui haru ära uuendada, siis ehitab.

jaanleppik commented 2 years ago

ei mina saa aru, mis ma valesti teen. uuendasin branchi koodi, ehitasin discampi, aga courseevents liste ei näe, saan 404 ikka http://localhost:4000/en/industry-courseevents-search http://localhost:4000/industry-courseevents-search

jaanleppik commented 2 years ago

industry local buildis saan /industry-courseevents-search aadressilt listi, discampi puhul ei saa

mpennaste commented 2 years ago

Ah kurja, DisCamp... sellega ma väga rohkem ei arvestanud kui et äkki tulevikus (kuna DisCampEvents on tühi), nagu kirjutasin, et jäi ebaselgeks. Samas jah, minu viga, et sai ka mainitud Industry listi aadressi juures. Kopeerisin Industry oma ja nüüd siis /discamp-courseevents-search peaks olema olemas seal ka listivaade.

jaanleppik commented 2 years ago

Ehitab. Muutsin http://localhost:4000/discamp-courseevents-search vaadet, aga sinna ei lae listi vaatesse pilte ära. Kasutan sama koodi piltide japks, nagu siin: https://discoverycampus.poff.ee/dc-events-search/?categories=Music%20Meets%20Film

mpennaste commented 2 years ago

... sinna ei lae listi vaatesse pilte ära.

Testisin, Strapis pole ühelgi DisCampi eventil pilte, mõnele testiks lisades laadis ära.

jaanleppik commented 2 years ago

aha, minu paha.

jaanleppik commented 2 years ago

discamp-events-search / discamp-courseevents-search Palun siia PUG'i filtritesse ka Festival edition Event Type (uus relatsioon uuele collectionile) Event Access (uus relatsioon uuele collectionile)

mpennaste commented 2 years ago

@LiisKasper Palun sinu abi järgnevate asjade Datamodelisse kirjutamisega (kui juba kusagil pole, ma ei leidnud) :)

Event Type (uus relatsioon uuele collectionile) Event Access (uus relatsioon uuele collectionile)

LiisKasper commented 2 years ago

Lisasin mudelisse EventType ja EventAccess ühtlasi ka #TagSecret Mis collectionid viitavad EventType ja EventAccess peale? Neid seoseid mudelis pole. Lisasn ka lifecycle starterid, et saaks kirjed kustutada jne.

mpennaste commented 2 years ago

Aitäh sulle :) Ma tean, et CourseEvents viitab EventType ja EventAccess'le, aga kas mõni collection veel, seda ei tea

jaanleppik commented 2 years ago

Buildin localis industry ja võtan lehe http://localhost:4000/industry-events-search klikates public=FALSE (OFF) courseEventile saan kirja 'Palun logi sisse' Saan aru, et see on näide sellises courseEventist, mille lehte ei saa üldse avada ilma, et oleks sisse logitud ja kasutajal oleks roll, mis lubab courseeventi näha. Kas sel juhul ei saaks suunata login lehele, nagu industry eventide puhul seni - http://industry.poff.ee/login ja peale sisse logimist suunata tagasi courseEventi lehele.

Aga esimeseks ma arvan, et peaks tegema sellise restricted lahenduse, nagu industry eventid seni on olnud - ainult video vaatamine nõuab sisse logimist ja vajalike õiguste olemasolu. Kui klikkan courseEventil, mille public=TRUE (ON) - siis saan courseEventi lehe lahti, aga video vaatamiseks pean Eventivaliga sisse logima. (mida ma oma localis teha ei saa). E selline variant, kus näen courseEventi lehte va neid osi, mille nägemine peaks nõudma sisselogimist ja õiguste olemasolu - võiks olla see restricted versioon - nagu pressi artikli puhul tegid :)

E kas peaksime tähistama / nimetama need 2 erinevat restricted versiooni: Restricted 1: kasutaja näeb lehte, aga mõned lehe osad näuavad kindlaid kasutajaõigusi Restricted 2: lehe avamine / nägemine nõuab kindlaid kasutajaõigusi

mpennaste commented 2 years ago

Kas sel juhul ei saaks suunata login lehele, nagu industry eventide puhul seni - http://industry.poff.ee/login ja peale sisse logimist suunata tagasi courseEventi lehele.

Saaks

Aga esimeseks ma arvan, et peaks tegema sellise restricted lahenduse, nagu industry eventid seni on olnud - ainult video vaatamine nõuab sisse logimist ja vajalike õiguste olemasolu.

Läheb eelnevaga vastuollu, aga kuna see on põhimõtteliselt elemendipõhine, siis saab teha, peaasi, et tervikuna asendatav element.

Kui klikkan courseEventil, mille public=TRUE (ON) - siis saan courseEventi lehe lahti, aga video vaatamiseks pean Eventivaliga sisse logima. (mida ma oma localis teha ei saa). E selline variant, kus näen courseEventi lehte va neid osi, mille nägemine peaks nõudma sisselogimist ja õiguste olemasolu - võiks olla see restricted versioon - nagu pressi artikli puhul tegid :)

See on see vana süsteem, st tahab sisselogimist, st see kõik tulebki ümber teha. Esialgu see layout jms ja siis tööle panna.

E kas peaksime tähistama / nimetama need 2 erinevat restricted versiooni: Restricted 1: kasutaja näeb lehte, aga mõned lehe osad näuavad kindlaid kasutajaõigusi Restricted 2: lehe avamine / nägemine nõuab kindlaid kasutajaõigusi

On avalik leht ja on "valitutele" leht - Kui pole õiguseid, näed vähe, kui on, näed rohkem.