Kalenteri: Tieto jos työntekijä muokannut omaa/mahdollisesti toisen kalenterimerkintää #5959
Conversation
kechpaja-at-gofore
changed the title
|
|
||
| ALTER TABLE calendar_event | ||
| ALTER COLUMN created_by SET NOT NULL, | ||
| ALTER COLUMN modified_by SET NOT NULL, |
There was a problem hiding this comment.
created_by ja modified_by eivät ole varsinaisesti tarpeellisia tähän muutokseen, mutta uskon, että niiden lisääminen osana tätä muutosta on hyvä future proofingin kannalta.
Voisin kuitenkin myös ottaa ne pois, jos ydintiimin jäsenet ovat vahvasti eri mieltä siitä.
There was a problem hiding this comment.
Onko noilla modified_at ja content_modified_at mikä semanttinen ero keskenään? Se tekninen updated_at on sitten vielä kolmantena. Näköjään toi erottelu on ollut jo aiemmin, mutta en keksi miksi. Tietääkö esim @tmkrepo ? Näyttäisi siltä että nykyään tuohon calendar_event tauluun tehdään vain inserttejä ja deletejä eikä koskaan updateja eli kumpikaan noista ei ole välttämätön tällä hetkellä (data on aina sama kuin created), mutta etenkään molempia ei tarvitse olla ellei ne päivity toisistaan riippumattomasti sen perusteella millainen update tehtiin.
There was a problem hiding this comment.
content_modified_at pyrkii indikoimaan valikoitujen tapahtuman alaisten elementtien (osallistujien tai keskusteluaikojen) uusimmasta käyttäjälähtöisestä muutoksesta. Käytetään käyttöliittymällä kertomaan viimeisimmän tapahtuman alaisen yksityiskohdan muutoshetki. Tuo on kokonaan manuaalisesti päivitettävä, eikä se välttämättä tarkoita suoraan muiden relaatiorivien uusinta leimaa. modified_at on suoraan riviin liittyvä muutos.
There was a problem hiding this comment.
Se mikä tästä pullasta ilmeisesti siis puuttuu on että kun tuo content_modified_at päivitetään niin pitäisi päivittää myös uusi content_modified_by.
There was a problem hiding this comment.
Niin oli tarkoitus tehdä. Onko se jäänyt toteuttamatta jossain?
There was a problem hiding this comment.
Ainakin CalendarEventQueries tiedostossa on kahdessa kohtaa UPDATE calendar_event joihin ei tainnut tässä diffissä tulla lisäystä. Tosin nuokin on molemmat funktioita, joissa päivitetään eventtiä itseään eikä sen lapsielementtejä. Esimerkiksi createCalendarEventAttendees jonka ymmärtäisin Tatun kuvauksen perusteella vaikuttavan sen parent eventin content_modified_at kenttään ei kuitenkaan käy päivittämässä sitä vaan ainoastaan inserttaa calendar_event_attendee tauluun. En tiedä kuinka tärkeää tietoa nuo on käyttöliittymän kannalta et voisiko vaan luopua, mutta nyt vaikuttaa vähän rikkinäiseltä.
There was a problem hiding this comment.
No muokkasin sen ainakin niin, että (content_)modified_by päivitetään aina silloin, kun (content_)modified_at päivitetään. Ainakin jos en ole missannut mitään.
There was a problem hiding this comment.
Tarkistin content_updated_at leimausten toimintaa ja ne päivittyvät nyt sekä calendarEventAttendee muutosten (updateCalendarEvent) että calendarEventTime muutosten (updateCalendarEventPeriod) yhteydessä. Molempiin noihin muutospaikkoihin on tässä PR:ssä lisätty myös muutoksen tekijä, joten vaikuttaisi olevan kunnossa.
kechpaja-at-gofore
requested review from
Joosakur,
akheron,
Gekkio and
reynders
…alenterimerkintää
kechpaja-at-gofore
force-pushed
the
calendar-metadata
branch
from
5c8709c to
6580f47
Compare
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
No description provided.