Skip to content

Cleanup API links - #673

Merged
jakub-id merged 1 commit into
mainfrom
cleanup-links
Jul 21, 2026
Merged

Cleanup API links#673
jakub-id merged 1 commit into
mainfrom
cleanup-links

Conversation

@jakub-id

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings July 21, 2026 12:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the broker API index (“/”) to remove non-actionable top-level links and replace the generic patron-requests link with two concrete, usable links for borrowing and lending requests.

Changes:

  • Remove eventsLink and locatedSuppliersLink from the API index response.
  • Replace patronRequestsLink with borrowingRequestsLink and lendingRequestsLink (including the required side query parameter).
  • Update OpenAPI schema and index handler tests to match the new index response shape.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
broker/test/api/api-handler_test.go Updates index response assertions to expect borrowing/lending patron request links instead of events/located suppliers.
broker/oapi/open-api.yaml Updates the Index.links schema: removes old link fields and requires the new borrowing/lending link fields.
broker/api/api-handler.go Adjusts the index handler to emit the new borrowing/lending links and drop the removed ones.

@jakub-id
jakub-id merged commit 33f2ca0 into main Jul 21, 2026
8 checks passed
@jakub-id
jakub-id deleted the cleanup-links branch July 21, 2026 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants