8000 sip2: fix patron language by jma · Pull Request #3707 · rero/rero-ils · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

sip2: fix patron language #3707

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 17, 2024
Merged

sip2: fix patron language #3707

merged 1 commit into from
Jun 17, 2024

Conversation

jma
Copy link
Contributor
@jma jma commented Jun 13, 2024
  • Fixes the language patron preference for sip2 messages.
  • Fixes missing sip2 error message for circulation error.

Co-Authored-by: Johnny Mariéthoz Johnny.Mariethoz@rero.ch
Co-Authored-by: Laurent Dubois laurent.dubois@itld-solutions.be
Co-Authored-by: Pascal Repond pascal.repond@rero.ch

@github-actions github-actions bot added the f: circulation Concerns the circulation interface or backend label Jun 13, 2024
@jma jma force-pushed the maj-fix-sip2-language branch from 52c4ff6 to 7df6d48 Compare June 13, 2024 11:26
@jma @lauren-d @PascalRepond
sip2: fix patron language
930cf9d 
* Fixes the language patron preference for sip2 messages.
* Fixes missing sip2 error message for circulation error.
* Closes rero#3423.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
Co-Authored-by: Laurent Dubois <laurent.dubois@itld-solutions.be>
Co-Authored-by: Pascal Repond <pascal.repond@rero.ch>
@jma jma force-pushed the maj-fix-sip2-language branch from 7df6d48 to 930cf9d Compare June 13, 2024 12:28
@jma jma requested a review from lauren-d June 13, 2024 12:29
@jma jma marked this pull request as ready for review June 13, 2024 12:29
@jma jma merged commit 9079774 into rero:staging Jun 17, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rerowep rerowep rerowep approved these changes

@lauren-d lauren-d lauren-d approved these changes

@PascalRepond PascalRepond PascalRepond approved these changes

Assignees

@jma jma

Labels
f: circulation Concerns the circulation interface or backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jma @rerowep @lauren-d @PascalRepond
0