ViesTIM-projektin (viestilistat, postilistat) jatkokehitys #2296

Open dezhidki opened 3 years ago

In GitLab by @totalund on Jun 9, 2021, 16:22

Tässä kortissa on oleellisimmat ViesTIM-projektin jatkokehitystarpeet, kuten projektin vaatimusmäärittelyssä tilassa Jatkokehitykseen olevat vaatimukset, järjestelmätestauksessa todetut virheet ja muut tarpeet, jotka ovat oleellisia viestijärjestelmän toimimiseen. Vaatimusmäärittelyssä on myös tiloissaja Ei projektissa olevia vaatimuksia, joita ei ole tähän korttiin erikseen vielä nostettu.

Vaatimusmäärittelyn Jatkokehitykseen-tilan tehtävät:

[ ] Viestilistalla on voitava olla useita omistajia (vaatimus 5.9).
[ ] TIMistä lähetettyjä yksittäisiä viestejä tulee voida arkistoida lähettäjän henkilökohtaiseen arkistoon (vaatimus 8.7).
[ ] TIMin tulee arkistoida kukin viesti vain kerran (vaatimus 8.11). Katso TODO mailman_events.py#L101.
[x] TIM-viestin lähettäjän on voitava valita TIM-viestin tärkeys (vaatimus 9.4). Katso TODO timMessage/routes.py#L304
[ ] TIM-viesti tulee voida näyttää valituilla sivuilla jokaiselle niitä katseleville (vaatimus 9.8).
[ ] TIM-viestin kohdesivujen URL-osoitteissa tulee voida käyttää regexiä (vaatimus 9.10).

Järjestelmätestauksessa (2. raportti) havaitut virheet, joita projektin aikana ei ehditty ratkaista:

[ ] Testitapaus 5.13: (Tarkennettava testissä käytettävät syötteet ja testatut tunnukset).

Projektin aikana koodiin kirjoitetut TODO-merkinnät (viittaavat ViesTIM-repoon):

[ ] Viestilistan jäsenien esitys aakkosjärjestyksessä. Katso TODO message-list-admin.component.ts#399 (TODO merkattu kahteen paikkaan, mutta järjestys kannattaa luonnollisesti tehdä vain yhdessä).
[ ] Virheenkäsittely / virheen esittäminen käyttäjälle. Katso TODO messagingUtils.ts#13.
[x] Muokkaa viestinlähetyskomponenttia niin, että se ei sulkeudu automaattisesti jos vastaanottajakenttä tyhjennetään käsin. Katso TODO tim-message-send.component.ts#L89.
[ ] Näytä mitä ryhmää TIM-viesti koskee. TODO tim-message.component.ts#L184.
[ ] Käsittele mailman-rest-events -kirjaston lähettämä SubscriptionEvent ja UnsubscriptionEvent. Katso TODO mailman_events.py#L96 ja mailman_events.py#L99
[ ] Sähköpostilistan kautta tullut viesti edelleenlähetetään muihin viestikanaviin. Katso TODO mailman_events.py#L129.
[ ] Käyttäjän asettamista yhteystiedoista asetetaan sähköpostilistalle sähköpostiosoitteet oikein. Katso TODO messagelist_utils.py#L707
[ ] Kun käyttäjiä lisätään ryhmään tai poistetaan ryhmästä, näiden viestikanavat päivitetään puskuroidusti. Katso TODO messagelist_utils.py#L745 ja messagelist_utils.py#L767.
[ ] Viestilistan viestiarkiston dokumentin "vain teksti"-asetuksen toteutus. Katso TODO messagelist_utils.py#L251.
[ ] Viestilistan arkistointitavan muuttuessa viestilistan jäsenet saavat ilmoituksen muutoksesta. Katso TODO messagelist/routes.py#L184.
[ ] Kun globaalisti näkyvät TIM-viestit on toteutettu, tarkasta käyttäjän oikeus nähdä dokumentti ennen kuin viesti näytetään. Katso TODO timMessage/routes.py#L110

Muut tehtävät (Viestilistat):

[x] TIM-käyttäjä voi lisätä itselleen muita sähköpostiosoitteita.
- Liittyy kortteihin #2295 ja #1986
[x] Autentikointi Postoriukseen TIMin kautta => estetään rekisteröinti suoraan
- Tämä vaatii pienen allauth-plugninin toteuttamisen. Aluksi riittää hyvin yksinkertainen menettely (ei tarvitse toteuttaa OAuth-autentikointipalvelua)
[x] Angular-käyttöliittymän lokalisointi
[x] Arkistoidun viestin näyttäminen Angular komponentilla suoraan. Tai oikeastaan komponentit TimArchiveHeader, TimArchiveFooter. Itse viesti saa olla normaalia "TIMiä", niin sitä voi kommentoida ja muokata
[x] Hallintakomponentin muuttaminen pluginiksi => siirretään pois pääbundlesta
[ ] TIM-käyttäjä voi määrätä itselleen sähköpostiosoitteistaan primääriosoitteen (primääriosoite on jakeluosoite, kaikki osoitteet ovat lähetysosoitteita).
[x] TIM-käyttäjän liittyessä viestilistalle (tai lisätessä / poistaessa osoitteitta) hänen viestilistansa päivittyvät automaattisesti.
[ ] Jos mailmaniin yhteys katkeaa, tulee joko estää kaikki viestilistojen ominaisuudet. Vaihtoehtoisesti tulee pystyä synkronoimaan tiedot, kun yhteys palautuu
[ ] Selaintestien laajentaminen (ilmoituksen näkyminen tai ei näkyminen tietyillä sivuilla; käyttäjien lisääminen, muokkaaminen ja poistaminen viestilistasta)
[ ] Lisättäessä jäsentä listalle sähköpostilla tarkastetaan, kuuluuko sähköposti TIM-käyttäjälle. Jos kuuluu, lisätään MessageListTimMember-objektina eikä ulkopuolisena jäsenenä.
[ ] Arkistossa olevaan viestiin vastaaminen TIMillä (voisi lisätä aikanaan Header-komponenttiin?)
[ ] Kun näytetään listan sivulla valittua ryhmää, voisi olla linkki ryhmän tietojen muokkaukseen
[x] Käyttöohjeen siirto muiden TIM-ohjeiden luokse. Linkki ohjeiden sopivan kohtaan itse ohjelmakoodissa. Ks mallia esim Tulota dokumentti
[ ] "Message lists-toiminnon alle: "Jätä sivulle TIM-viesti" ja silloin on automattisesti sivun URL-kohdekentässä ja ruksit kun TIM-viestin jättämisessä pitää olla.
[x] listoja saa tehdä nykyisin vain Group Admin. Mutta vastaavasti jos on oikeus Sisu-ryhmään, niin pitäisi sille ryhmälle saada tehdä lista (ks edellä)
[ ] Käyttäjän kutsuminen viestilistalle. Katso TODO messagelist/routes.py#L288. Katso lisäksi TODO emaillist.py#L338 (todennäköisesti TODOssa mainitun lipun voi suoraan asettaa arvolle True ja kutsuminen hoidetaan muilla keinoilla).
- Myös käyttäjän poistuminen suoraan TIM-käyttöliittymästä
[ ] #3148 ryhmän nimi ja jäsäniksi ko ryhmä. Tässä tavassa voisi sallia listan tekeminen, vaikka ei olisi ryhmänteko-oikeutta.
- Eli listan nimeksi ryhmän nimi (oletusehdotus).
- oletuksena vain jäsenet saavat lukea arkistoa
- kuvaukseen että ryhmän xxx lista
- listan jäseniksi se ryhmä josta lisääminen aloitettiin
[x] Indikaattori, kun lisätään jäsenet listaan (erityisesti, jos lisätään iso ryhmä)
[x] Kun uudet jäsenet on lisätty, näytetään ilmoitus, että sivu pitää päivittää => tai sitten päivitetään taulukko automaattisesti
[x] Saapuneet viestit voisi konvertoida markdowniksi tietyissä osissa
- Linkkien ympäröinti < ja > -merkkeihin (saa niistä klikattavat)
- Lainauksien oikea käsittely
- Korjataan eur03.safelinks-linkit
[ ] Jotkut paremmat oletukset subscribe/unsubscribe -notifikaatioille
- Linkki TIM-hallintasivulle
- Mahdollisesti listan tiedot (esim. lukumäärä)
[ ] Mahdollisuus konfiguroida viesti, joka lähetetään liittymisestä/poistumisesta
[ ] jos kurssille tekee postilistoja, ne voisi näkyä kuten kurssin ryhmät vasemman reunan menussa (tunnistaisikohan ne siitä että joku kurssin ryhmistä on postilistassa)
[ ] arkistossa voisi olla ettei punaista lukumerkintää tule headeriin eikä footeriin jos saa pienellä vaivalla
[ ] Dokumenttilistaus TimTableksi
- Voi suodattaa kaikki sarakkeet
- Voi valita näytettävät sarakkeet
- Voi säätää "Last edited"-sarakkeen formattia
[ ] Jotkin spostilistan asetukset ovat väärin laitettu arkistointivälilehteen
[ ] Jos lisättävään ryhmään ei ole oikeutta, pitää näyttää virhe
[ ] Pitäisikö riittää vain view-oikeus ryhmän lisäämiseksi viestilistalle?
[ ] Jäsenlistauksen laajentaminen
- TimTable pohjana
- Voisi näyttää jäsenten lukumäärä (kaikki jäsenet + ryhmät erikseen)
- Erillinen näkymä, jossa näytetään kaikki jäsenet ja omassa sarakkeessa lista ryhmistä, johon hän kuuluu
[ ] Bounce-asetus voi olla TIM-käyttöliittymässä
[ ] Jos käyttäjä on JYU-käyttäjä, niin viestilistaan laitetaan automaattisesti lähetysoikeus kaikkiin mahdollisiin osoitteisiin:
- matti.meikalainen@jyu.fi
- matti.meikalainen@student.jyu.fi
- matmeika@jyu.fi (osoittena käyttäjätunnus)
- matmeika@student.jyu.fi

Muut tehtävät (TIMin sisäiset viestit):

[ ] Ilmoitus saapuneesta viestistä sähköpostitse
[ ] Viestin otsikko tottelemaan markdownia
[x] TIM-viestin jättämisen käyttöliittymä ei tarvitse resetoida viestin lähettämisen jälkeen
[x] viestien lista ei ole aikajärjestyksessä
[ ] viestien listassa voisi näkyä mikä on kuitattu
[ ] Viestin peruuttaminen viestidokumentissa
[ ] Viestin kohdedokumenttien näyttäminen viestidokumentissa
[ ] Tieto viimeisestä lukukerrasta (eli milloin viimeksi viesti näytettiin) tallentumaan kantaan
[ ] Viestin poiskuittaus vain viestin yhteyteen laitetulla "salasanalla". Käyttötarkoitus: on kysely johon pakko vastata. Ko "salasana" laitetaan kyselyn loppuun ja sitten kun sen sieltä bongaa, voi kuitata viestin pois. Henkilökohtainen salasana hieman haastava, eli aluksi yksi yhteinen (joka heikentää tehoa, koska sen voi kysyä kaverilta).
[ ] Viestit alikansioihin (millä perustein?)
[ ] Viestien lähetyskomponentti voisi käyttää tavallista lohkoeditoria Silloin viestit voi kirjoittaa Markdownilla ja muuntaa Plaintext/HTML -muotoon lähetystä varten
[ ] Käyttöliittymä uuden TIM-viestin lähettämiseen samalle joukolle TIM-viestidokumentista
[ ] Mahdollisuus generoida suurempia tilastoja lukeneista ja ei-lukeneista (esim. tunnistetaan, kuka ei ole vastannut pitkään aikaan)
[ ] Lukeneiden/lukemattomien määrä suoraan TIM-viestikansion ja/tai viestidokumentin näkymään
[ ] TableForm queryn kopiointipainike sivun valikkoon eikä vastauskomponenttiin
[ ] Viestin perustiedot näkyviin (tärkeys, umpeutumisaika)
[ ] Mahdollisuus muuttaa umpeutumisaika ja tärkeys viestin luomisen jälkeen
[ ] Mahdollisuus lähettää TIM-viestejä automaattisesti tietyille sivuille, kun jollekin tietylle viestilistalle saapuu viesti

Vikoja:

[x] Käyttäjien mergen yhteydessä tulee varoitus Mailmanilta
[ ] Jos lähettäjän nimessä on skandit, lähettäjän tiedot menevät sekaisin (häviää spostiosite, tulee ylimääräinen "haamulähettäjä")
[ ] Listan omistajien/jäsenten päivitys ei välity oikein arkistokansion oikeuksiin
[x] Next message ei näy Footerissa
[x] TimTablea odotellessa archives-alla olevat järjestetään id:n mukaisesti (tulisi aikajärjestys)
[x] arkiston hakemistossa [viesti21] jokaisen viestin edessä ei oikeastaan tuo lisäarvoa? Eli Jos se on Titlessä, sen voisi hakemistolistauksessa riisua pois
[ ] listan muutosviestit voisi tulla "digestinä". Nyt sain 22 postin ryöpyn ja jos pistää 300 opiskelijan kurssille, niin tulee aika monta postia.
[x] ei toteutetut valinnat voisivat olla tekstiltäälkin harmaita niin niitä ei vahingossa kokeilisi ruksia päälle/pois
[x] tein listan ohj21c. Oikeudet menivät väärin, ks alla
[x] Listan sivulla Archive policy: on otsikkona, mutta sen alla olvat ruksit eivät varmaan kuulu tuolle otsikolle. Minkäs takia arkistointitapaa ei saanutkaan muuttaa?
- Arkistointitapaa ei saanut muuttaa, koska sekä muuttamisen toteuttavaa logiikkaa ei ole toteutettu että muutoksesta jäsenille ilmoittavaa logiikkaa ei ole toteutettu -TL.
- [ ] Sivulta https://tim.jyu.fi/view/messagelists/ohj2k23 pääsee pois ilman varoituksia tallentamatta
[ ] URLin syöttäminen TIM-viestien lähetyksessä käsin ei toimi, koska parseri syö /-merkit pois (ks. esim. #3324)

In GitLab by @vesal on Jun 14, 2021, 23:11

Tein listan: https://tim.jyu.fi/view/messagelists/ohj21c ja pyysin arkistointitavaksi Members only. Yksittäisessä viestissä oikeus on kuitenkin Anonymous ja hakemistossa ei kellään. Tässä tapauksessa ymmärtääkseni oikeus pitäisi olla molempiin ryhmällä ohj21c

In GitLab by @roeyskoe on Jun 15, 2021, 09:30

Viesteissä olevat linkit olisi mukava olla arkistossakin klikattavina linkkeinä, vaikka ne eivät raakaviestissä sellaisia olisikaan. Kaikki sähköpostiohjelmat kuitenkin tekevät teksti -> klikattava muunnoksen automaattisesti.

In GitLab by @vesal on Jun 15, 2021, 09:34

sellaisia olisikaan. Kaikki sähköpostiohjelmat kuitenkin tekevät teksti -> klikattava muunnoksen automaattisesti.

Voisikos tämä jopa olla Pandoc-muunnoksessa oleva ominaisuusa että aina jos on linkki, joka ei ole linkki (tai hipsuissa) olisi linkki?

Vesa

In GitLab by @roeyskoe on Jun 15, 2021, 09:41

Minun mielestä kyllä. Harvemmin tulee kuitenkaan sellaista tilannetta linkin ei haluttaisi olevan klikattavissa.

In GitLab by @vesal on Jun 15, 2021, 09:49

Tuolla olisi:

pandoc -f markdown+autolink_bare_uris

jolla tuon saisi ehkä ilman koodia?

In GitLab by @totalund on Jun 16, 2021, 20:03

Koodia katsellessa näyttää siltä, että hakemiston oikeuksia ei ainakaan tule asetettua. Tuo Anonymous on kuitenkin jännä tässä kontekstissa. Jossain kohtiin projektin aikana kompastuin omaan näppäryyteeni tuon arkistoinnin ja indeksoinnin kanssa, joten mahtaakohan kyseessä olla sellainen probleemi. Täällä näyttää kyllä eri kohdassa olevan virhe oikeuksien asetuksen kanssa, mutta tämän virheen ei pitäisi tuohon Members only -juttuun vaikuttaa.

Tutkin asiaa.

In GitLab by @totalund on Jun 19, 2021, 20:03

Merkitsin projektin aikana koodiin jätetyt TODO-merkinnät omiksi tehtävikseen ja viittaan niihin ViestTIM-repon linkeillä, niin niiden konteksti ei mene vahingossa muuttumaan. Projektin sovellusraportissa voidaan TODO-merkinnöistä sitten viitata suoraan tähän korttiin.

marked the task tein listan ohj21c. Oikeudet menivät väärin, ks alla as completed

marked the task Angular-käyttöliittymän lokalisointi as completed

marked the task Hallintakomponentin muuttaminen pluginiksi => siirretään pois pääbundlesta as completed

marked the task Autentikointi Postoriukseen TIMin kautta => estetään rekisteröinti suoraan as completed

marked the task Käyttöohjeen siirto muiden TIM-ohjeiden luokse. Linkki ohjeiden sopivan kohtaan itse ohjelmakoodissa. as completed

marked the task listoja saa tehdä nykyisin vain Group Admin. Mutta vastaavasti jos on oikeus Sisu-ryhmään, niin as completed

marked the task Listan sivulla Archive policy: on otsikkona, mutta sen alla olvat ruksit eivät varmaan kuulu tuolle as completed

marked the task TIM-käyttäjän liittyessä viestilistalle (tai lisätessä / poistaessa osoitteitta) hänen viestilistansa päivittyvät automaattisesti. as completed

marked the task Arkistoidun viestin näyttäminen Angular komponentilla suoraan. Tai oikeastaan komponentit TimArchiveHeader, as completed

marked the task Next message ei näy Footerissa as completed

marked the task Indikaattori, kun lisätään jäsenet listaan (erityisesti, jos lisätään iso ryhmä) as completed

marked the task Kun uudet jäsenet on lisätty, näytetään ilmoitus, että sivu pitää päivittää => tai sitten päivitetään taulukko automaattisesti as completed

marked the task TimTablea odotellessa archives-alla olevat järjestetään id:n mukaisesti (tulisi aikajärjestys) as completed

marked the task ei toteutetut valinnat voisivat olla tekstiltäälkin harmaita niin niitä ei vahingossa as completed

marked the task Dokumenttilistaus TimTableksi as completed

marked the task Dokumenttilistaus TimTableksi as incomplete

marked the task arkiston hakemistossa [viesti21] jokaisen viestin edessä ei oikeastaan tuo lisäarvoa? Eli Jos se on Titlessä, as completed

marked the task Saapuneet viestit voisi konvertoida markdowniksi tietyissä osissa as completed

marked the task Käyttäjien mergen yhteydessä tulee varoitus Mailmanilta as completed

In GitLab by @vesal on Sep 23, 2021, 11:03

Kirjoitusvirhe: Lukumerkintää voi peruuttaa viestikansiossa

Sellainen huono, että kun jätän TIM-viestin, niin akikki asetukset nollautuvat ja jos haluan vain vaihtaa vastaanottajaryhmää, niin kaikki pitää tehdä uudelleen. Voisi jäädä tuohon kaikki ne asetukset jotka olin tehnyt

tim-viesti ruksittuna
sivut sen mukaan mitä otin
mitä muita rukseja laitoin

ainakin niina kauan kunnes virkistän sivun.

[ ] Viestin sisällön voi kirjoittaa TIM-viestiin MarkDownina, mutta otsikko ei tottele.
[ ] Viestien katseluss aviestit eivät ole aiakjärjestyksessä
[ ] kuitatut viestit voisivat olla nimetty jollakin nimellä josta näkee että kuitattu (koska ylimääräistä saraketta ei saada helpolla)

marked the task TIM-viestin lähettäjän on voitava valita TIM-viestin tärkeys (vaatimus 9.4). Katso TODO timMessage/routes.py#L304 as completed

marked the task TIM-viestin jättämisen käyttöliittymä ei tarvitse resetoida viestin lähettämisen jälkeen as completed

marked the task viestien lista ei ole aikajärjestyksessä as completed

marked the task TIM-käyttäjä voi lisätä itselleen muita sähköpostiosoitteita. as completed

marked the task Muokkaa viestinlähetyskomponenttia niin, että se ei sulkeudu automaattisesti jos vastaanottajakenttä tyhjennetään käsin. Katso TODO tim-message-send.component.ts#L89. as completed

unassigned @dezhidki

Kopioitu #3146:

[ ] Miten löytää helposti kursille luomansa postilistan? Kurssilaisten listaan on kotisivulla linkki.

Sivulta https://tim.jyu.fi/view/messagelists/ohj2k23 pääsee pois ilman varoituksia tallentamatta

TIM-JYU / TIM

ViesTIM-projektin (viestilistat, postilistat) jatkokehitys #2296