Skip to content

Update OpenAPI schema#24

Merged
kantacky merged 2 commits intomainfrom
update-openapi-schema
Mar 27, 2026
Merged

Update OpenAPI schema#24
kantacky merged 2 commits intomainfrom
update-openapi-schema

Conversation

@dotto-api-schema-updater
Copy link
Copy Markdown
Contributor

This PR updates the OpenAPI schema from dotto-typespec.

Triggered by: fun-dotto/dotto-typespec@a0afda6

@kantacky kantacky self-assigned this Mar 27, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Admin BFF and external Academic API OpenAPI specifications (and regenerated Go code) to align with the latest dotto-typespec changes, including request/parameter shape updates for timetable-related endpoints.

Changes:

  • Remove dayOfWeek query parameter from TimetableItemsV1_list (spec + generated binding/params).
  • Add required roomIds to TimetableItemRequest (specs + regenerated Go models).

Reviewed changes

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

File Description
openapi/openapi.yaml Updates BFF OpenAPI contract: removes dayOfWeek filter; requires roomIds in timetable item create request.
openapi/external/academic_api/openapi.yaml Updates upstream Academic API spec: requires roomIds in timetable item create request.
generated/external/academic_api/client.gen.go Regenerates upstream Academic API Go client models to include RoomIds.
generated/api.gen.go Regenerates BFF server/client types and binding to reflect removal of dayOfWeek and addition of RoomIds.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kantacky kantacky merged commit 5e3fdfa into main Mar 27, 2026
9 checks passed
@kantacky kantacky deleted the update-openapi-schema branch March 27, 2026 09:11
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.

2 participants