KohaSuomi / Koha

(KOHA) Koha is a free software integrated library system (ILS). Koha is distributed under the GNU GPL version 3 or later. Note: this is a synced mirror of the official Koha repo.
https://koha-community.org/
GNU General Public License v3.0
7 stars 13 forks source link

Asiakkaan Ilmoitukset-välilehden Toimitushuomautus-sarakkeeseen oikea tieto epäonnistumisen syystä #329

Open ajrajala opened 1 year ago

ajrajala commented 1 year ago

Kehitysehdotukseni

Tällä hetkellä asiakkaan Ilmoitukset-sivulla näkyy sarake delivery_note (jota ei kait pysty piilottamaan näkyvistä Taulujen asetukset -sivulla), johon tulee epäonnistuneista lähetyksistä ainoana tietona teksti "Virhe sähköpostia lähetettäessä".

Olisi kätevää, jos tuon nykyisen tiedon sijaan käyttäjä näkisikin oikean failure_coden silloin kun lähetys on epäonnistunut. (Vaskissa virkailijoita varten on tällä hetkellä tehtynä erillinen raportti, jolla failure_coden voi tarkistaa kun selvitetään syytä tekstiviestilähetyksen epäonnistumiselle.)

Jotain muuta?

No response

AnneliO commented 1 year ago

Samaa asiaa Redmine-tiketissä 5572.

pknuuti commented 1 year ago

Emmin kommentti Redminessä: Kyseessä on tavallaan ohjelmavirhe. Kun kannasta katsoo, kyseisen asiakkaan viestissä lukee failure_code-sarakkeessa "Recipient phone number is invalid or unknown". Kohan koodissa kuitenkin käsitellään nämä failure_codet seuraavasti:

[% IF ( QUEUED_MESSAGE.failure_code ) %] [% IF ( QUEUED_MESSAGE.failure_code == "INVALID_BORNUMBER" ) %]Invalid borrowernumber [% borrowernumber | html %] [% ELSIF ( QUEUED_MESSAGE.failure_code == 'NO_EMAIL' ) %]Unable to find an email address for this borrower [% ELSIF (matches = QUEUED_MESSAGE.failure_code.match('INVALID_EMAIL:(\w+)') ) %]Invalid [% matches.0 | html %] email address found [% borrowernumber | html %] [% ELSIF ( QUEUED_MESSAGE.failure_code == 'NO_FROM' ) %]Missing from email address [% ELSIF ( QUEUED_MESSAGE.failure_code == 'MISSING_SMS' ) %]Missing SMS number [% ELSIF ( QUEUED_MESSAGE.failure_code == 'DUPLICATE_MESSAGE' ) %]Message is duplicate [% ELSIF ( QUEUED_MESSAGE.failure_code == 'NO_NOTES' ) %]No notes from SMS driver [% ELSIF ( QUEUED_MESSAGE.failure_code == 'SENDMAIL' ) %]Unhandled email failure, check the logs for further details [% ELSIF ( QUEUED_MESSAGE.failure_code == "UNKNOWN_ERROR" ) %]Unknown error [% ELSE %]Error occurred while sending email. [% END %] [% END %]

Koska "Recipient phone number is invalid or unknown" ei vastaa yhtäkään yllä olevista if-elsif-ehdoista eikä failure_codea tulosteta suoraan Kohaan, Koha käyttää else-ehdon "Error occurred while sending email.". Viesti saataneen kyllä Kohaan, mutta se kuinka se tehdään niin, että viesti on käännettävissä voi olla hieman monimutkaisempi toteuttaa.

"Laitan tämän kommentoitu tilaan, katsellaan jossain välissä paremmin mitä tälle tehdään."

Suljen Redminen tiketin.

AnneliO commented 9 months ago

Koha-Suomen asiantuntijaryhmä 5.2.2024: Ei edistetä itse, tehdään tiketti yhteisön Bugzillaan.

ajrajala commented 8 months ago

Varmistan vielä ennen kuin teen Bugzillaan tiketin, että onko varsinainen tarve siis se että tuolla asiakkaan ilmoituksissa failure_code -käsittelyn ehtoihin joko pystyisi lisäämään sieltä nyt puuttuvia ehtoja tai sitten delivery note -sarake näyttäisi suoraan failure_coden?

ajrajala commented 7 months ago

Huomasinkin, että tuo yllä oleva kysymys ei varmaan @emta001 tavoittanut, joten tägäänpä Emmin vielä tähän ketjuun.

emta001 commented 7 months ago

Varmistan vielä ennen kuin teen Bugzillaan tiketin, että onko varsinainen tarve siis se että tuolla asiakkaan ilmoituksissa failure_code -käsittelyn ehtoihin joko pystyisi lisäämään sieltä nyt puuttuvia ehtoja tai sitten delivery note -sarake näyttäisi suoraan failure_coden?

Sanoisin, että ensimmäinen vaihtoehto on se lähemmin oikea mitä tässä haetaan.

ajrajala commented 7 months ago

Yhteisöön tehty tiketti https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36487.