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

Takaajan tiedoissa näkyy taattavan julkisoikeudelliset maksut #128

Closed
pirliis opened this issue Apr 2, 2024 · 14 comments
Closed

Takaajan tiedoissa näkyy taattavan julkisoikeudelliset maksut #128

pirliis opened this issue Apr 2, 2024 · 14 comments
Assignees
Labels
bug Something isn't working

Comments

@pirliis
Copy link

pirliis commented Apr 2, 2024

Mikä vikana?

Meillä on intranetusercss-rimpsu:
/* Piilota Näytä takaajalle lainat ja Näytä takaajalle maksut -tiedot asiakkaan Tiedot-sivulta */
#patron-privacyguarantor + li { display: none; }
#patron-privacyguarantor { display: none; }

Rimpsu piilottaa tällä hetkellä takaajan tiedoista taattavan yksityisoikeudelliset maksut, mutta ei julkisoikeudellisia maksuja.

OUTIssa taattavan borrowernumber=162771 maksut:
image

Näkymä takaajan borrowernumber=64487 tiedoissa:
image

Samoin, takaajalla näkyy välilehti "Taattavat maksut":
image

Mitä pitäisi tapahtua

CSS-rimpsun tulisi piilottaa takaajalta sekä taattavan julkis- että yksityisoikeudelliset maksut.

Kuinka toistaa ongelma/asia

No response

Jotain muuta?

No response

Asiasanat/tagit

No response

@pirliis pirliis added the bug Something isn't working label Apr 2, 2024
@emta001
Copy link
Contributor

emta001 commented Apr 3, 2024

Välilehden saa piiloon lisäämällä tämän:

#guarantees_finesandcharges-tab { display: none; }

Mutta tuo viesti onkin sitten vähän hankalampi, koska siinä ei ole id:tä johon napata.

@emta001 emta001 self-assigned this Apr 3, 2024
@emta001
Copy link
Contributor

emta001 commented Apr 3, 2024

Kyllä tuon viestinkin sai piiloon, kun otti vaan silmän käteen ja huomas, että sillä on class 😄 Sen saa siis piiloon tällä:

.guarantees-fines { display: none; }

@atarnus
Copy link

atarnus commented Apr 3, 2024

Näillä CSS-pätkillä voi piilottaa halutut kohteet:

/* Piilota Taattavien maksut -välilehti */
#guarantees_finesandcharges-tab { display: none; }

/* Piilota taattavien maksut asiakassivun taattavat-listasta */
#pat_moremember li.guarantees-fines { display: none; }

Jos joissain kimpoissa ei näitä haluta piilottaa, niin sitten tulee miettiä, että millä korjataan näkymä todenmukaiseksi. Muutetaanko tekstejä kuvaamaan että näkyvät maksut on julkisoikeudellisia vai halutaanko molemmat maksut näkyviin.

Emmi näemmä ehtikin ensin jo vastaamaan. :)
Onkohan meillä kimppoja, joissa tämä pelkkä piilotus ei riitä?

@atarnus atarnus assigned atarnus and unassigned emta001 Apr 3, 2024
@AnneliO
Copy link
Contributor

AnneliO commented Apr 3, 2024

Nuo aloituksessa mainitut rimpsut piilottavat Tiedot-sivulta nämä rivit:

kuva

Niiden ei pitäisi piilottaa varsinaisesti mitään maksuja (ellei vahingossa niin tee). Mutta noilla Emmin ja Kassun rimpsuilla sekin tosiaan onnistuu. :)

@pirliis
Copy link
Author

pirliis commented Apr 19, 2024

OUTIssa halutaan näkyviin taattavan maksut takaajan tiedoissa. Olisiko mahdollista saada takaajan Tiedot-sivulle, jossa taattavan maksu näkyvät, vaikka samanlaisen tekstin kuin on varsinaisesti asiakkaan tiedoissa maksuista, jotta siinä näkyy myös maksujen yhteissumma?

Eli nyt takaajan tiedoissa näkyy taattavan maksut näin:
image

Voisiko tekstin muuttaa näin:
"Asiakkaan taattavalla on lainauskieltoon vaikuttavia maksuja €5.00. Yhteensä (sis. maksut, jotka eivät vaikuta lainauskieltoon): €6.00.

Muutosta ei tarvitse toteuttaa ennen versionvaihtoa, jos ei ehditä. Tai jos tekstimuutos on mahdoton toteuttaa, OUTI tyytyy nykyiseen tekstiin, vaikka se antaa vähän väärää tietoa. :)

@AnneliO
Copy link
Contributor

AnneliO commented Apr 19, 2024

OUTIssa halutaan näkyviin taattavan maksut takaajan tiedoissa. Olisiko mahdollista saada takaajan Tiedot-sivulle, jossa taattavan maksu näkyvät, vaikka samanlaisen tekstin kuin on varsinaisesti asiakkaan tiedoissa maksuista, jotta siinä näkyy myös maksujen yhteissumma?

Eli nyt takaajan tiedoissa näkyy taattavan maksut näin: image

Voisiko tekstin muuttaa näin: "Asiakkaan taattavalla on lainauskieltoon vaikuttavia maksuja €5.00. Yhteensä (sis. maksut, jotka eivät vaikuta lainauskieltoon): €6.00.

Muutosta ei tarvitse toteuttaa ennen versionvaihtoa, jos ei ehditä. Tai jos tekstimuutos on mahdoton toteuttaa, OUTI tyytyy nykyiseen tekstiin, vaikka se antaa vähän väärää tietoa. :)

Se toinen muutos vaati koodimuutoksia, joten pelkkä tekstimuutos käännöstiedostoihin ei taida tässäkään riittää. Teetkö tästä toiveesta ihan erillisen kehitysehdotuksen, jos pääkäyttäjäpalaverissa löytyy muitakin ajatusta kannattavia? :)

@atarnus
Copy link

atarnus commented Apr 19, 2024

Jos ei muuta tehdä, niin voi tuota tekstiä muokata vaikka rimpsulla sanomaan että nuo on lainauskieltoon vaikuttavia maksuja. Tällöin olisi ainakin se annettu tieto oikein, jos nyt jotain tietoa puuttuu.

Kannattaa huomioida siihen toiseen tikettiin myös se, että halutaanko korjata samalla tuo taattavan maksut -välilehti. Sekään ei näytä kaikkia maksuja eriteltynä.

taattavan-maksut

@AnneliO
Copy link
Contributor

AnneliO commented Apr 19, 2024

Luulen, että koodimuutosta odotellessa voisi tuon "Asiakkaan taattavalla on yhteensä maksuja €5.00." muuttaa po-tiedostoissakin muotoon "Asiakkaan taattavalla on lainauskieltoon vaikuttavia maksuja yhteensä €5.00.". Vastaava muutos sopi tieteellisille, joten en näe siihen estettä tässäkään tapauksessa. Siitäkin voisi tehdä erillisen tiketin, niin pysyy erilaiset muutokset paremmin hallinnassa. :)

@pirliis
Copy link
Author

pirliis commented Apr 19, 2024

Okei. Teen siis kaksi erillistä tikettiä. :)

@ReettaSiili
Copy link

Piilotin Siilin nextillä taattavien maksut ja viestit. Toimii!

@Kati-Helle
Copy link

Välilehden nimeksi Taattavan maksut ?
kuva

@AnneliO
Copy link
Contributor

AnneliO commented Apr 23, 2024

Välilehden nimeksi Taattavan maksut ? kuva

Teetkö tästä erillisen tiketin? :)

@AnneliO
Copy link
Contributor

AnneliO commented May 3, 2024

Dokumentoitava.

@AnneliO AnneliO assigned AnneliO and unassigned atarnus May 3, 2024
@pirliis
Copy link
Author

pirliis commented May 16, 2024

Tämän tiketin muutosehdotuksista on tehty omat tiketit:
KohaSuomi/Koha-translations#46
KohaSuomi/Koha#1173
#141

Suljen tämän tiketin.

@pirliis pirliis closed this as completed May 16, 2024
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