KohaSuomi / Bugiton

Bugi-perjantaiden tikettien hallinnointiin tarkoitettu repositorio
0 stars 0 forks source link

Asiakkaan Lainaus- ja Tiedot- välilehdelle näkyviin kaikkien maksujen summa #9

Open lmstrand opened 1 year ago

lmstrand commented 1 year ago

Tukipyyntö

Virkailijaliittymään halutaan näkyviin Lainaus- ja Tiedot- välilehdille asiakkaan kaikkien maksujen summa (julkisoikeudelliset + ei-julkisoikeudelliset). Tällä hetkellä vain julkisoikeudelliset maksut näkyvät kyseisissä näkymissä.

Jotain muuta?

No response

lmstrand commented 1 year ago

Viety testeille, branchissa ksdev/ks-0122-on-0032-KD-G762-include-all-charges-for-totals-in-intranet.

Kati-Helle commented 1 year ago

Helle-testissä testatessani vaikuttaisi toimivalta ratkaisulta :)

Asiakkaalla ID=23345 maksuja seuraavasti:

Asiakkaan Tiedot -näytöllä näkyy julkisoikeudelliset maksut sekä kaikkien maksujen kokonaismäärä kuva

Lainaus-näytöllä näkyy julkisoikeudelliset maksut sekä maksujen kokonaismäärä kuva

(Kokonaismaksumäärän selite englanniksi: Total (incl. non-blocking) )

iinaniemi commented 1 year ago

Näyttää toimivan oikein myös Vaski-testissä. :)

pknuuti commented 1 year ago

Myös Vaara-testissä toimii kuten pitääkin.

ReettaSiili commented 1 year ago

Siilin testi ei nyt jostain syystä osaa erottaa julkisoikeudellisia ja yksityisoikeudellisia maksuja toisistaan: kuva

Tuotannossa erottelu mielestäni toimii, eli näyttää tässä vain julkisoikeudelliset: kuva

lmstrand commented 1 year ago

Tein siili-testille itselleni uuden kortin hinta 10e ja muovikassiostos oli 4e. Tiedoilla näkyy: image

Kati-Helle commented 1 year ago

Testattavana oleva ratkaisu on mielestäni hyvä. Tiedoissa näkyy nyt selkeästi julkisoikeudelliset eli Kohasta automaattisesti muodostuneet maksut, joilla on vaikutusta maksuista johtuvaan lainauskieltorajaan. Lisäksi näkyy asiakkaan maksujen kokonaismäärä (julkisoikeudelliset maksut + yksityisoikeudelliset maksut).

ReettaSiili commented 1 year ago

Ok, eli ratkaisu sinänsä toimii siili-testilläkin, mutta jotkut määrittelyt on nyt pielessä. Redmine-tiketissä #5409 on määritelty näin:

set_account_types_siili.sql

update account_debit_types set is_system=1 where code in ('1. huomautuskirje', '2. huomautuskirje', 'ODUE', 'OVERDUE', 'RESERVE_EXPIRED'); update account_credit_types set is_system=1 where code in ('1. huomautuskirje', '2. huomautuskirje', 'ODUE', 'OVERDUE', 'RESERVE_EXPIRED');

update account_debit_types set is_system=0 where code in ('NEW_CARD', 'MANUAL', 'LOST'); update account_credit_types set is_system=0 where code in ('NEW_CARD', 'MANUAL', 'LOST');

Mutta myöhemmin tarkentamaani lisäystä ei ole varmaan lisätty:

kuva

lmstrand commented 1 year ago

Yhteisössä on asiasta tiketti, jossa sattumoisin toivotaan maksujen näkyvän juuri testillä näkyvällä tavalla: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33228

Yritetään viedä ominaisuus yhteisöön.

AnneliO commented 1 year ago

Tägään tähän tikettiin vielä tiketin KohaSuomi/Koha#378

AnneliO commented 1 year ago

Jos asiakkaalla on pelkästään ei-julkisoikeudellisia maksuja, ne eivät näy nyt mulla ollenkaan lainaus-sivulla. Miten muilla?

pirliis commented 1 year ago

Jos asiakkaalla on pelkästään ei-julkisoikeudellisia maksuja, ne eivät näy nyt mulla ollenkaan lainaus-sivulla. Miten muilla?

Ei näy. Ja Tiedot-sivulla ainoastaan Maksut-välilehdellä.

Kati-Helle commented 1 year ago

Ei näy Helle-testissäkään Lainaus- eikä Tiedot-sivulla. Yksityisoikeudelliset, ei lainauskieltorajaan vaikuttavat maksut näkyvät lainauksen jälkeen (asiakasID 6911). kuva

Seliteteksti 'Asiakkaalla on maksamattomia maksuja varauksista, lainoista, tms: 3,28' kaivannee muutosta.

Asiakkaan maksutiedot kuva

lmstrand commented 1 year ago

Testeille viety korjaus maksujen näkymiseen, jos asiakkaalla on esim. vain ei-julkisoikeudellisia maksuja.

Kati-Helle commented 1 year ago

Helle-testillä näkyvät nyt Lainaus-sivulla ei-julkisoikeudelliset maksut. Asiakkaalla (ID 6911) on testatessa vain ei-julkisoikeudellisia maksuja .

kuva

ajrajala commented 1 year ago

Vaski-testilläkin näkyy maksut oikein Lainaus- ja Tiedot -sivuilla, jos asiakkaalla vain ei-julkisoikeudellisia maksuja.

AnneliO commented 1 year ago

Koha-Suomen palaveri 14.9.2023: Näihin muutoksiin liittyen pitää tehdä uudet po-tiedostot. Lasse tekee.

pirliis commented 1 year ago

OUTIssa näyttäisi toimivan myös nyt maksujen näkyminen oikein.

Toimiiko julkis- ja yksityisoikeudellisten maksujen erottelu jo Finnassa meidän spekseillä?

Kati-Helle commented 1 year ago

Helle-testi asiakas ID 34638, jonka maksut ovat yhteensä 38,34 koostuen näistä

Helle-Finna-testiin sisäänkirjauduttua tämä ilmoitus kuva

Helle-Finna-testin Maksut-välilehden näkymää kuva

AnneliO commented 1 year ago

Toimiiko julkis- ja yksityisoikeudellisten maksujen erottelu jo Finnassa meidän spekseillä?

Tuosta pitänee tehdä erillinen tiketti (jos ei jo ole), koska tässä tiketissä säädetään vain Kohan näkymää.

pirliis commented 1 year ago

Toimiiko julkis- ja yksityisoikeudellisten maksujen erottelu jo Finnassa meidän spekseillä?

Tuosta pitänee tehdä erillinen tiketti (jos ei jo ole), koska tässä tiketissä säädetään vain Kohan näkymää.

En löytänyt vielä omaa tikettiä, joten tehdään oma. Mutta ajattelin, että pitäisikö meillä keskustella ensin pääkäyttäjäpalaverissa yhdessä, miten maksut halutaan Finnassa asiakkaille jatkossa näkyvän. Tehdään sitten yhteisen päätöksen mukainen tiketti. :)

AnneliO commented 1 year ago

"Seliteteksti 'Asiakkaalla on maksamattomia maksuja varauksista, lainoista, tms: 3,28' kaivannee muutosta."

@Kati-Helle minkälaista muutosta tässä ajattelit?

Kati-Helle commented 1 year ago

"Seliteteksti 'Asiakkaalla on maksamattomia maksuja varauksista, lainoista, tms: 3,28' kaivannee muutosta."

@Kati-Helle minkälaista muutosta tässä ajattelit?

Alla kuva aiemmasta tiketti-kommentistani. Tässä tapauksessa asiakkaalla on vain yksityisoikeudellisia maksuja (3Dtulostus) eikä Kohasta automaattisesti muodostuneita julkisoikeudellisia maksuja (eli ei varausmaksuja eikä myöhästymismaksuja) kuva

Nythän tilanne on tuon osalta muuttunut, kun julkisoikeudelliset maksut ja yksityisoikeudelliset maksut on eritelty Huomio-kohdassa kuva

Kati-Helle commented 1 year ago

Lainausnäkymää, kun asiakkaalla (ID 27767) on vain julkisoikeudellisia maksuja. Nyt puuttuu sininen palkki tekstillä: Asiakkaalla on maksamattomia maksuja varauksista, lainoista, tms: [maksumäärä] kuva

Lisään tähän kommenttiin näin jälkikäteen vielä lainausnäkymän asiakastiedosta, kun asiakkaalla on sekä yksityisoikeudellisia että julkisoikeudellisia maksuja. Tässäkin tapauksessa puuttuu sininen palkki tekstillä: Asiakkaalla on maksamattomia maksuja varauksista, lainoista, tms: [maksumäärä] kuva

IrinaHalminen commented 1 year ago

Jos ja kun kaikki maksut saadaan näkyville tuohon Huomion alle, onko tuo sinisen palkin teksti edes tarpeellinen?

Kati-Helle commented 1 year ago

Tarpeettomalta vaikuttaa maksujen näkymisuudistusten myötä.

iinaniemi commented 1 year ago

Minusta myös tuo sinisen laatikon teksti on huono. Siinä puhutaan maksuista kuten varaukset ja lainat, vaikka kyseessä on nimenomaan virkailijan käsin lisäämät, lainauskieltoon vaikuttamattomat maksut kuten tulostusmaksut. Eli kannatan sitä että joko sininen laatikko kokonaan pois, tai sitten siinä voisi lukea vain tyyliin "asiakkaalla on maksamattomia maksuja €x,xx"

AnneliO commented 1 year ago

Sinisen laatikon teksti on englanniksi

The patron has unpaid charges for holds, rentals etc of %s

Ja se on tosiaan käännetty suomeksi

Asiakkaalla on maksamattomia maksuja varauksista, lainoista, tms: %s

Vuokriahan meillä ei ole, niin tuossa on varmaan aikanaan yritetty "suomettaa" tekstiä. Jos ei haluta suoraa suomennosta edelleenkään, niin sitten varmaan parasta on jättää varaukset ja vuokrat tosiaan kokonaan pois.

AnneliO commented 1 year ago

Kokeilen myös, saako sinisen palkin piiloon, jolloin ei tarvitsisi tehdä puutteellista suomennosta.

Kati-Helle commented 1 year ago

Minusta myös tuo sinisen laatikon teksti on huono. Siinä puhutaan maksuista kuten varaukset ja lainat, vaikka kyseessä on nimenomaan virkailijan käsin lisäämät, lainauskieltoon vaikuttamattomat maksut kuten tulostusmaksut. Eli kannatan sitä että joko sininen laatikko kokonaan pois, tai sitten siinä voisi lukea vain tyyliin "asiakkaalla on maksamattomia maksuja €x,xx"

Kaipaisiko tiedon siitä, että kyseessä on yksityisoikeudelliset maksut? Esim. Asiakkaalla on maksuja, jotka eivät vaikuta lainauskieltoon: €x,xx Asiakkaalla on maksamattomia maksuja, jotka eivät vaikuta lainauskieltoon: €x,xx

Tai joku muu yhtenäinen tapa tässä näkyvän yksityisoikeudellisen maksutiedon kanssa kuva

emta001 commented 1 year ago

Tämä on tällä hetkellä yhteisössä Failed QA. Testaussuunnitelma puuttuu.

Kati-Helle commented 1 year ago

Maksut näkyvät nyt Helle-tuotannossakin oikein asiakkaan

Lainaus-näytöllä kuva

Tiedot-näytöllä kuva

(Koha-päivitys 17.10.2023)

iinaniemi commented 1 year ago

Näkyy nyt oikein myös Vaskissa

pknuuti commented 1 year ago

Vaarassakin tuotannossa oikeanlainen maksutieto.

AnneliO commented 1 year ago

@pirliis ehtisitkö jossain välissä päivittää tämän muutoksen myös Kohan ohje suomeksi -ohjeeseen? :)

pirliis commented 11 months ago

Päivitin Maksut-sivun alkuun julkis- ja yksityisoikeudellisista maksuista. Saa kommentoida, jos haluatte jotain lisää tai muutosta.

MariaJoon commented 11 months ago

Lapin testiltä näyttäisi puuttuvan kokonaan asetus ManInvInNoissuesCharge? Kannasta kyllä löytyy.

AnneliO commented 11 months ago

Lapin testiltä näyttäisi puuttuvan kokonaan asetus ManInvInNoissuesCharge? Kannasta kyllä löytyy.

Testikannoissa on tällä hetkellä yhteisöstä tuotu muutos, joka poistaa tuon asetuksen ja korvaa sen toiminnolla, jossa maksutyypeissä voi valita tyyppikohtaisesti, lasketaanko ne mukaan lainarajaan. Tuo ominaisuus tulee meille käyttöön varsinaisesti seuraavassa versionvaihdossa.

Kati-Helle commented 11 months ago

Helle-testissä testatessani vaikutti toimivan oikein. Myös Finna-testissä toimi oikein.

MariaJoon commented 11 months ago

Testikannoissa on tällä hetkellä yhteisöstä tuotu muutos, joka poistaa tuon asetuksen ja korvaa sen toiminnolla, jossa maksutyypeissä voi valita tyyppikohtaisesti, lasketaanko ne mukaan lainarajaan. Tuo ominaisuus tulee meille käyttöön varsinaisesti seuraavassa versionvaihdossa.

No niinpä olikin, menin jo sekaisin näiden tikettien kanssa.

piiaseme commented 9 months ago

En ole varma onko tämä tälle asialle oikea tiketti, mutta kokeillaan.. OUTIn asiakkaalla: https://outi.koha-suomi.fi/cgi-bin/koha/circ/circulation.pl?borrowernumber=35982 on myöhästymismaksu 0,30e, joka näkyy hänellä näin: kuva

Mistä tämä kumma johtuu, kun sen pitäisi näkyä näin: Asiakkaalla on maksamattomia maksuja €0,30. Yhteensä (sis. maksut, jotka eivät vaikuta lainakieltoon): €0,30.

AnneliO commented 9 months ago

En ole varma onko tämä tälle asialle oikea tiketti, mutta kokeillaan.. OUTIn asiakkaalla: https://outi.koha-suomi.fi/cgi-bin/koha/circ/circulation.pl?borrowernumber=35982 on myöhästymismaksu 0,30e, joka näkyy hänellä näin: kuva

Mistä tämä kumma johtuu, kun sen pitäisi näkyä näin: Asiakkaalla on maksamattomia maksuja €0,30. Yhteensä (sis. maksut, jotka eivät vaikuta lainakieltoon): €0,30.

Luulen, että tämä liittyy jotenkin siihen, että OUTIssa on ManInvInNoissuesCharge -järjestelmäasetuksessa vielä valittuna "Sisällytä" muokattuja debit-maksuja, kun lasketaan yhteen asiakkaan maksuja noissuescharge järjestelmäasetuksen summaan. Se huomautus ei nyt osaa erotella julkisoikeudellisia ja yksityisoikeudellisia maksuja, niin se näyttää ne noin. Teillä taisi olla työn alla tuon järjestelmäasetuksen muuttaminen toiseen asentoon. :)

piiaseme commented 9 months ago

En ole varma onko tämä tälle asialle oikea tiketti, mutta kokeillaan.. OUTIn asiakkaalla: https://outi.koha-suomi.fi/cgi-bin/koha/circ/circulation.pl?borrowernumber=35982 on myöhästymismaksu 0,30e, joka näkyy hänellä näin: kuva Mistä tämä kumma johtuu, kun sen pitäisi näkyä näin: Asiakkaalla on maksamattomia maksuja €0,30. Yhteensä (sis. maksut, jotka eivät vaikuta lainakieltoon): €0,30.

Luulen, että tämä liittyy jotenkin siihen, että OUTIssa on ManInvInNoissuesCharge -järjestelmäasetuksessa vielä valittuna "Sisällytä" muokattuja debit-maksuja, kun lasketaan yhteen asiakkaan maksuja noissuescharge järjestelmäasetuksen summaan. Se huomautus ei nyt osaa erotella julkisoikeudellisia ja yksityisoikeudellisia maksuja, niin se näyttää ne noin. Teillä taisi olla työn alla tuon järjestelmäasetuksen muuttaminen toiseen asentoon. :)

Suurimmalla osalla OUTIn asiakkaista on summa molemmissa kohdissa. Tämä on ensimmäinen kohtaamamme tapaus, jolla summa on vain yhteensä-kohdassa.