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

Tietueen varaukset sivulle näkyviin asiakkaan kirjastokortin numero #97

Closed
MikkoLiimatainen opened this issue Jun 20, 2023 · 8 comments

Comments

@MikkoLiimatainen
Copy link

Kehitysehdotukseni

Tietueen varaussivulla näkyy kirjastokortin lukemisen jälkeen teksti "Asiakas: Kirjaston asiakas Turku, Pääkirjasto".
Koha request pl varauksen tiedot asiakas
Tähän näyttäisi sivupohjassa request.tt vaikuttavan arvo no_cardnumber => 1
Github request tt no_cardnumber
Varsinainen teksti tuohon tulee tuon patron-title.inc sivun kautta ja siellä tuo no_cardnumber asettaa display_cardnumber arvoksi 0.
Github patron-title inc display_cardnumber
Display_cardnumber puolestaan vaikuttaa siihen, mikä teksti tuohon varaussivulle tulee.
Github patron-title inc Teksti varaussivulle
Nyt kun display_cardnumber asetetaan no_cardnumber-arvon avulla nollaksi, tulee patron-title.inc sivulta default-teksti. Jos no_cardnumber => 1 jätettäisiin request.tt-sivulta pois tuosta kohtaa, jäisi patron-title.inc:n display_cardnumber arvoksi yksi ja silloin varaussivulle tulostuvan tekstin pitäisi olla asiakkaan kortin numero.

Jos tulkintani koodin toiminnasta on oikea, niin tämä vaikuttaisi olevan yksinkertainen muutos, joka toisi erittäin suuren hyödyn varausten tekemiseen.

Jotain muuta?

No response

@pknuuti
Copy link

pknuuti commented Jun 20, 2023

Tieto asiakkaasta olisi hyvä saada näkyville, varmistaisi varauksen teon oikealle henkilölle.

@AnneliO
Copy link
Contributor

AnneliO commented Jul 13, 2023

Tähän liittyy HidePatronName-järjestelmäasetus, joka piilottaa asiakastiedot perustiedot-näytöllä lainattujen niteiden kohdalta ja muuttaa varaustenteko-sivulla nimen tilalle "Kirjaston asiakas...".

Perustiedot-näytöllä nimen tilalle tulee kirjastokortin numero, mikä olisi looginen tapa myös varauksenteko-sivulla.

@AnneliO
Copy link
Contributor

AnneliO commented Jul 28, 2023

Tämä näyttäisi vaikuttavan myös ilmoitukseen, jossa kerrotaan varauksia olevan jo maksimimäärä:

kuva

kuva

@MikkoLiimatainen
Copy link
Author

Juu, tuo no_cardnumber näyttää tosiaan olevan pitkin tuota reserve/request.tt sivupohjaa.
Varauksen korttinumero no_cardnumber asetus varoituksissa
Minusta ei korttinumeron piilottamiseen tuolla sivulla ole mitään syytä, joten nuo voisi kaikki poistaa.
Noiden poistamisen jälkeen sivun pitäisi toimia niin, että tuolla HidePatronName-asetuksella voi kontrolloida näkyykö sivulla asiakkaan nimi, vai pelkkä korttinumero. Käytännössä siis samalla tavalla kuin tietueen perustiedot näytöllä lainattujen niteiden kanssa.

@AnneliO
Copy link
Contributor

AnneliO commented Nov 20, 2023

Versiossa 23.05.05 tulee muutos, jossa asiakkaan kirjastokortin numero lisätään varauksentekosivulle. Sandboxissa tehdyn testauksen mukaan, kirjastokortin numero näkyy, vaikka päälle laittaa HidePatronName-järjestelmäasetuksen.

Bugi 34457

Siirrän tiketin version 23.x-tietovarantoon ja testattavaksi, kunhan saamme nextit käyttöön.

@AnneliO AnneliO transferred this issue from KohaSuomi/Koha Nov 20, 2023
@Kati-Helle
Copy link

Helle-Nextillä asiakkaan kirjastokorttiarvo näkyy Asiakas-kentässä
Image

@pknuuti
Copy link

pknuuti commented Jan 24, 2024

Vaaran nextillä testattu, sama tulos kuin Hellessä eli asiakaskentässä näkyy kortin numero, mikä on hyvä juttu.

@ajrajala
Copy link

Varuilta lisään vielä tähän, että testasin myös usean nimekkeen varaamista yhdellä kertaa, silloinkin korttinumero näkyvissä. 👍

@AnneliO AnneliO closed this as completed Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Suljettu
Development

No branches or pull requests

5 participants