annapiir/prosse

Työprosessien seurantasovellus

Prosse - Työprosessien seurantasovellus

Osoite: https://prosse.herokuapp.com/

Testitunnukset:

• Käyttäjätunnus: testi
• Salasana: testi

Kuvaus

Sovelluksen avulla voidaan määritellä erilaisista tehtävistä koostuvia työprosesseja sekä seurata näiden etenemistä.

Organisaatiossa on työprosesseja, jotka koostuvat tehtävistä. Sovelluksessa määritellään tehtäviä, muodostetaan tehtävistä työprosesseja ja seurataan prosessien etenemistä tehtäväkohtaisesti.

Prosessista vastaava henkilö määrittelee tehtäviä ja kokoaa niistä työprosesseja. Hän myös määrittää tehtäville tekijät. Lisäksi prosessivastaava saa järjestelmältä raportteja työn etenemisestä ja tehtäviin sekä työprosessiin käytetystä ajasta.

Työntekijät merkkaavat käynnistetyistä prosesseista tehtävät aloitetuiksi ja lopetetuiksi ja voivat myös kommentoida työprosessikierrokseen liittyviä tehtäviä. Koska työntekijä voi tehdä samanaikaisesti töitä usean työprosessin parissa, hän saa järjestelmältä nähtäväkseen raportin, jossa on tietoja hänen tehtävistään (tehdyistä, tekeillä olevista ja hänelle osoitetuista).

Toimintoja:

• Kirjautuminen
• Käyttäjien lisääminen/poistaminen/muuttaminen
• Tehtävien määrittely/poistaminen/muuttaminen
• Työprosessien kokoaminen tehtävistä
• Tekijöiden määrittely työprosessiin kuuluville tehtäville
• Prosessiin kuuluvien tehtävien aloittaminen/lopettaminen
• Prosessivastaavan raportit
• Työntekijän raportit

Dokumentaatio

• Asennusohje
• Käyttöohje
• Tietokantakaavio (Create table -lauseet)
• Käyttäjätarinat
• Mitä jäi tekemättä?

prosse's Issues

Koodikatselmointi #1

Koodikatselmointi #1

Projekti haettu 11:30 13.08.2018.

Palaute

Projekti on hyvällä mallilla ja toteuttaa annettua hyvän rakenteen ja koodauksen mallia erinomaisesti. Applikaation käyttö on selkeää ja suoraviivaista. Toivottavasti tehtävien yhdistäminen prosesseiksi onnistuu jatkossa yhtä hyvin! :)

Pikku vikoja, helppo korjata

• Prosessi-sivulla sarakkeen nimenä on 'Viimeksi muokattu', vaikka kyseessä taitaa olla prosessin alku- ja loppuajat. Sarakkeet ovat kuitenkin helppo muokata /templates/prosessi/list.html -tiedostosta, ja kyseessä on vain pieni käyttisasia.

• Ylläolevaan liittyen, osassa näkymistä on ehkä hieman ylimääräisesti näkyvissä luontipäivä ja muokkaustietoa. Jos ne eivät ole olennaisia näkymälle, ehkä esim. juuri työprosessilistauksessa, ne tekevät sivusta helposti tarpeettoman sekavan. Tämä on kuitenkin pitkälti kiinni käyttötarpeesta, ja sittenkin makuasia. :)

• /auth/forms.py tiedoston kolmannesta luokasta (class KayttajaLisaysLomake(FlaskForm)) puuttuu csrf = False -määrittely.

• Käyttäjänlisäys-sivulla syötteen validointi ei toimi, esim. käyttäjänimi voi olla vain yhden merkin mittainen. Validointisäännöt on määritelty, mutta /auth/views.py -tiedostosta puuttuu kaavakkeen tietojen validointikutsu. Tämä on kuitenkin tehty oikein esim. /tehtavat/views.py -tiedostossa, joten eikun sieltä mallaamalla korjaamaan.

• Myöskään työprosessin nimeä ei validoida - joskin tämä on nähtävästi tarkoituksella kommentoitu ulos /prosessi/views.py -tiedostossa riveillä 61 ja 62..? Oletettavasti toiminnallisuuden haluttu toteutus on siis tuossa kohden hieman kesken - parempi näin että kokonaisuus kuitenkin toimii.

• Ylimääräiset init.py -tiedostot jokaisessa olioalihakemistossa kummastuttavat. Nämä voinee siivota pois huoletta.

Loppukommentti

Projekti on edennyt vaaditun mukaisesti ja tästä on varmasti hyvä jatkaa eteenpäin. Tsemppiä, menestystä ja hyvää lukuvuoden alkua!

Vertaisarviointi #2

Testasin sovellusta ja tässä muutama kommentti siitä:

• Prosessiin kun lisää tehtävän, niin tehtävä on aloitettu ja valmis defaulttina.
• Työprosesseista tulee internal server erroria joten en päässyt sitä testaamaan.
• Jotkut kentät näyttää Nonea esim. https://prosse.herokuapp.com/prosessitehtava/2 sarake "Kommentoija". Nämä yleensä lasketaan bugeiksi

Koodista:

• Aika paljon kommentteja jotka lähinnä selittävät mitä seuraavalla rivillä tehdään. Nämä ovat turhia sillä asia tulee ilmi koodista. Usein käy myös niin, että kommentit unohtuvat ja koodi muuttuu. Tällöin kommentti johtaa vain harhaan. Parempi siis jättää kokonaan pois.

• ORM mahdollistaa myös joinit, eli voisit varmaankin tehdän tämän sijasta:

    plista = Prosessi.query.all()
    prosessit = {}
  
    for p in plista:
        omistaja = Kayttaja.query.get(p.owner_id)
        prosessit[omistaja.tunnus] = p
  

  jotain tän tyylistä:

    plista = Prosessi.query.join(Kayttaja).all()
  

  vähemmän koodia :-)

• Kts esim. prosessitehtava_prosessi_nayta. Tässä tosiaan voisi yhdistää paljon jo kannan tasolla ja koodi yksinkertaistuisi paljon. Funktio on myös järkyttävän pitkä. Huomasin, että sulla ei taida olla yhtään funktiota koodissa joka ei olisi HTTP-käskyn käsittelijä. Materiaali mainitsee tästä huonosti ja ehkä tämä on muiden kurssien asiaa, mutta oikeasti kaikkea yhden käskyn aikana tapahtuvaa toimintaa ei tarvitse sulloa yhteen funktioon.

Tässä tärkeimmät joita kannattaa miettiä. Todella hienosti olet ottanut tän haltuun tässä lyhyessä ajassa. Paikkaat paljon kannan toiminnallisuutta omalla koodilla mikä on turhaa, mutta muuten tää on hieno sovellus ja mikä tärkeintä, se toimii. Lisäksi positiivista on, että sovellus on hyvin organisoitu ja jakaa toiminnallisuuden selkeisiin moduuleihin.