Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Järjestelmäasetus: DebarmentsToLiftAfterPayment - ksdev/ks-0006-bug-16223 #49

Closed
emta001 opened this issue Feb 9, 2023 · 19 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@emta001
Copy link
Collaborator

emta001 commented Feb 9, 2023

Siirrettävä ominaisuus

Koha-Suomen versiossa on aiemmin ollut käytössä järjestelmäasetus DebarmentsToLiftAfterPayment. Yhteisöön on kuitenkin tullut uusi ominaisuus "Patron restrictions/Asiakas rajoitukset(?)", jolla voidaan määritellä käytettäväksi erityyppisiä rajoituksia. Maksurajoja on ehdotettu yhteisöön osaksi ominaisuutta ja järjestelmäasetuksesta voidaan luopua.

@emta001 emta001 added the bug Something isn't working label Feb 9, 2023
@emta001 emta001 self-assigned this Feb 9, 2023
@emta001
Copy link
Collaborator Author

emta001 commented Feb 10, 2023

Tämä on viety vaski-nextille testattavaksi.

@AnneliO
Copy link
Contributor

AnneliO commented Feb 17, 2023

Onko Lift after payment ja Fee limit jostain erityisestä syystä pakollisia?

@AnneliO
Copy link
Contributor

AnneliO commented Feb 17, 2023

Testaa näin:

  1. Mene Ylläpito -> Asiakkaan rajoitukset / Administration -> Patron restrictions
  2. Tee uusi rajoitetyyppi tai muokkaa olemassaolevaa, esim. Manual.
  3. Valitse Lift after payment -kohtaan Yes
  4. Kirjoita Fee limit -kohtaan haluamasi maksuraja, esim. 5.00. Huom! Desimaalierottimena käytetään pistettä.
  5. Tallenna
  6. Mene jonkin testiasiakkaan tietoihin ja lisää hänelle sen tyyppinen rajoite, jota juuri käsittelit
  7. Lisää asiakkaalle maksuja yli rajoitteen maksurajan, jos maksuja ei ole ennestään.
  8. Huomioi, että rajoite on edelleen olemassa.
  9. Merkitse kaikki asiakkaan maksut maksetuksi ja tarkista, että rajoite on poistunut.
  10. Toista kohtia 7-9 erilaisilla maksumäärillä ja varmista, että rajoite ei poistu, jos maksuja jää jäljelle yli asetetun rajan ja että rajoite poistuu, kun maksujen määrä alittaa/saavuttaa määritetyn rajan. Huomioi, että kun maksuja on tismalleen rajoitteeseen määritetty määrä, rajoite poistuu. Kokeile myös maksujen poistoa.

@emta001
Copy link
Collaborator Author

emta001 commented Feb 17, 2023

Onko Lift after payment ja Fee limit jostain erityisestä syystä pakollisia?

Oot ensimmäinen joka tuosta huomauttaa. Eihän niiden tarttis olla, mikä lie ajatus mulla siinäkin ollut taustalla :D
Voi ne vaihtaakin, ainakin sen Fee limitin vois ottaa veke. Sehän on turha silloin kun toinen on No 🤔

@ajrajala
Copy link

Testasin sekä maksujen maksamista että poistoa ja rajoite käyttäytyi kuten piti.

Vaskissa ei näköjään ole koskaan ollut käytössä järjestelmäasetusta DebarmentsToLiftAfterPayment. 😄 Siksi nyt siis vähän tyhmä kysymys, mutta onko tämä toiminut siis niin, että rajoite maksuista on muodostunut asiakkaille automaattisesti kun määritelty maksuraja on tullut täyteen?

@AnneliO
Copy link
Contributor

AnneliO commented Mar 15, 2023

emta001

Onko Lift after payment ja Fee limit jostain erityisestä syystä pakollisia?

Oot ensimmäinen joka tuosta huomauttaa. Eihän niiden tarttis olla, mikä lie ajatus mulla siinäkin ollut taustalla :D Voi ne vaihtaakin, ainakin sen Fee limitin vois ottaa veke. Sehän on turha silloin kun toinen on No 🤔

Epäpakollisuus ois pop, varsinkin jos kimpassa ei haluta käyttää tätä toiminnallisuutta. Pakollisuus pakottaa kirjoittamaan kenttään kuitenkin jotain. :)

ajrajala

Testasin sekä maksujen maksamista että poistoa ja rajoite käyttäytyi kuten piti.

Vaskissa ei näköjään ole koskaan ollut käytössä järjestelmäasetusta DebarmentsToLiftAfterPayment. 😄 Siksi nyt siis vähän tyhmä kysymys, mutta onko tämä toiminut siis niin, että rajoite maksuista on muodostunut asiakkaille automaattisesti kun määritelty maksuraja on tullut täyteen?

OUTIssakaan tätä ei ole ollut käytössä, mutta luulisin, että rajoite on lisätty erikseen, eikä se ole muodostunut automaattisesti.

@emta001
Copy link
Collaborator Author

emta001 commented Mar 16, 2023

emta001

Onko Lift after payment ja Fee limit jostain erityisestä syystä pakollisia?

Oot ensimmäinen joka tuosta huomauttaa. Eihän niiden tarttis olla, mikä lie ajatus mulla siinäkin ollut taustalla :D Voi ne vaihtaakin, ainakin sen Fee limitin vois ottaa veke. Sehän on turha silloin kun toinen on No thinking

Epäpakollisuus ois pop, varsinkin jos kimpassa ei haluta käyttää tätä toiminnallisuutta. Pakollisuus pakottaa kirjoittamaan kenttään kuitenkin jotain. :)

Tätä pitää hetki tuumata, jos sen laittaa ei-pakolliseksi, jääkö se sitten herkästi täyttämättä? Pitäisi varmaan yrittää saada säädettyä tuota jotenkin niin, että jos Lift after payment on Yes, niin Fee limit on silloin pakollinen täyttää. Tai sitten siinä on oletuksena 0.

@emta001
Copy link
Collaborator Author

emta001 commented Mar 17, 2023

Laitoin yhteisöön patchin, jossa kumpikaan kentistä ei ole pakollinen, mutta jos "Lift after paymentin" laittaa päälle, "Fee limitistä" tulee pakollinen. En vielä tuonut nexteille, koska skeman rakennuksessa on vielä pieniä ongelmia.

@Kati-Helle
Copy link

Hellessä ei ole käytössä asetus DebarmentsToLiftAfterPayment.

@emta001
Copy link
Collaborator Author

emta001 commented Mar 17, 2023

Laitoin yhteisöön patchin, jossa kumpikaan kentistä ei ole pakollinen, mutta jos "Lift after paymentin" laittaa päälle, "Fee limitistä" tulee pakollinen. En vielä tuonut nexteille, koska skeman rakennuksessa on vielä pieniä ongelmia.

Tämä on nyt nexteillä testattavana.

@ajrajala
Copy link

Testasin asetusta ja jos Lift after paymentin laittoi päälle, Fee limit -kentän viereen tuli tieto pakollisuudesta eikä tallentaminen onnistunut jos kenttä oli tyhjä. Testasin myös uudestaan rajoituksen automaattista poistumista silloin, kun maksut maksetaan maksurajan alle, toimi edelleen kuten piti.

(Suomenkielisellä puolella molemmat kentät näkyi vielä pakollisena eli testasin enkuksi.)

@AnneliO
Copy link
Contributor

AnneliO commented Mar 29, 2023

Lisätyt rajoitteet eivät jostain syystä näy, kun asiakkaalle lisää rajoitusta:
kuva

@emta001
Copy link
Collaborator Author

emta001 commented Mar 29, 2023

Lisätyt rajoitteet eivät jostain syystä näy, kun asiakkaalle lisää rajoitusta: kuva

Works on my machine. Tutkaillaan.

@emta001
Copy link
Collaborator Author

emta001 commented Mar 29, 2023

OUTIssa ei ollut asetus PatronRestrictionTypes päällä. Nyt on ja pitäisi rajotusten näkyä OK.

@AnneliO
Copy link
Contributor

AnneliO commented Mar 29, 2023

Haa, niinpä tietenkin. Eipä tuollainen muistunut mieleen. :D Kiitos!

@pknuuti
Copy link

pknuuti commented Apr 4, 2023

Vaarassa on ollut käytössä DebarmentsToLiftAfterPayment-asetus. Meillä oli muutaman vuoden käytössä vuodenvaihteessa tuleva lainauskielto, jos asiakkaalla oli maksuja maksamatta. Kun asiakas sitten maksoi maksut vaikka verkkokirjaston kautta, tämä toiminto poisti asiakkaalle asetetun rajoituksen heti pois. Nyt tuota lainauskieltoa ei enää tule ja entisetkin rajoitukset on poistettu. Tuskin tuolle asetukselle on enää käyttöä?

@emta001
Copy link
Collaborator Author

emta001 commented Apr 5, 2023

Vaarassa on ollut käytössä DebarmentsToLiftAfterPayment-asetus. Meillä oli muutaman vuoden käytössä vuodenvaihteessa tuleva lainauskielto, jos asiakkaalla oli maksuja maksamatta. Kun asiakas sitten maksoi maksut vaikka verkkokirjaston kautta, tämä toiminto poisti asiakkaalle asetetun rajoituksen heti pois. Nyt tuota lainauskieltoa ei enää tule ja entisetkin rajoitukset on poistettu. Tuskin tuolle asetukselle on enää käyttöä?

Tuskin ellette halua lisätä asiakkaille rajoituksia manuaalisesti, jotka sitten poistuu kun maksuja ei enää ole tai ne on alle tietyn summan.

@Tuomas-Kyyti
Copy link

Vaikuttaa toimivan Kyytin nextissä ok, vaikka emme tällä tietoa ole ottamassa sitä käyttöön. Rajoite poistuu sekä maksamalla että maksun poistamalla.

Meillä on ollut käytössä DebarmentsToLiftAfterPayment-asetus, johon on määritetty että vuosittaisista lainauskieltoajoista tullut rajoite poistuu, kun maksut on maksettu. Lainauskieltoajossa siis kaikille, joilla oli maksuja, tuli tietty sama rajoiteteksti, joka sitten määritettiin tuossa DebarmentsToLiftAfterPayment-asetuksessa. Ihan hyödyllinen toiminnallisuus, johon varmaan voi jatkossa hyvin käyttää tätä uutta toimintoa.

Nyt meillä ei ole enää moneen vuoteen ollut lainauskieltoajoja, joten DebarmentsToLiftAfterPayment-asetuksen poistumisesta ei aiheutune ongelmaa. Eikä jäljellä olevia rajoitetekstejä lainauskieltoajoista tarvitse konvertoida tähän uuteen toimintoon.

@emta001
Copy link
Collaborator Author

emta001 commented Apr 24, 2023

Tämä toiminee kuten pitää. Suljen tiketin tältä erää.

@emta001 emta001 closed this as completed Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Suljettu
Development

No branches or pull requests

6 participants