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

MCBFF-37: Check-in API #41

Merged
merged 10 commits into from
Jan 13, 2025
Merged

MCBFF-37: Check-in API #41

merged 10 commits into from
Jan 13, 2025

Conversation

OleksandrVidinieiev
Copy link
Contributor

Purpose

Create a proxy endpoint for mod-circulation's Check-in API.
MCBFF-37

Approach

How does this change fulfill the purpose?

TODOS and Open Questions

  • Use GitHub checklists. When solved, check the box and explain the answer.

Learning

Describe the research stage. Add links to blog posts, patterns, libraries or addons used to solve this problem.

@OleksandrVidinieiev OleksandrVidinieiev merged commit eb1135f into master Jan 13, 2025
6 checks passed
@OleksandrVidinieiev OleksandrVidinieiev deleted the MCBFF-37 branch January 13, 2025 10:17
roman-barannyk pushed a commit that referenced this pull request Jan 16, 2025
* MCBFF-37 Declare check-in endpoint in module descriptor

* MCBFF-37 Declare check-in endpoint in circulation-bff.yaml

* MCBFF-37 Controller, service and API test

* MCBFF-37 Check-in client, extend request & response schemas, API tests

* MCBFF-37 Remove public access modifier

* MCBFF-37 Add 400

* MCBFF-37 Forward circulation check-in errors

* MCBFF-37 Add permission set

* MCBFF-37 Add dependency on interface `circulation`

* MCBFF-37 Remove unused imports

(cherry picked from commit eb1135f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants