Skip to content

Conversation

@flomonster
Copy link
Member

@flomonster flomonster commented Jan 8, 2026

Dependency:

Within this PR:

  • All train schedule related endpoints are removed
  • Remove a legacy cli command
  • Regenerate openapi and rtk files
  • Remove models and tables with migration

@github-actions github-actions bot added area:editoast Work on Editoast Service kind:api-change labels Jan 8, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

⚠️ API changes

This Pull Request introduces some changes in the API:

  • please own it: notify or even prepare dedicated PR(s) to consumer projects

@flomonster flomonster force-pushed the fam/remove-train-schedules branch from 1ede4c6 to 915d424 Compare January 8, 2026 13:07
@github-actions github-actions bot added the area:front Work on Standard OSRD Interface modules label Jan 8, 2026
@flomonster flomonster changed the title editoast: remove train schedule endpoints and unused cli Cleanup train schedules endpoints models 🧹 Jan 8, 2026
@flomonster flomonster force-pushed the fam/remove-train-schedules branch 5 times, most recently from 94b7142 to beba8ed Compare January 9, 2026 11:00
@flomonster flomonster changed the base branch from dev to cni/adapt-paced-train-schema-6 January 9, 2026 11:00
@clarani clarani force-pushed the cni/adapt-paced-train-schema-6 branch from 451610c to 5007ae7 Compare January 9, 2026 11:43
@flomonster flomonster force-pushed the fam/remove-train-schedules branch from beba8ed to 24295a8 Compare January 9, 2026 14:08
Base automatically changed from cni/adapt-paced-train-schema-6 to dev January 12, 2026 13:10
@flomonster flomonster force-pushed the fam/remove-train-schedules branch from b87d8b8 to d66b49f Compare January 13, 2026 11:32
- All train schedule related endpoints are removed
- Remove a legacy cli command
- Regenerate openapi and rtk files

Signed-off-by: Florian Amsallem <[email protected]>
@flomonster flomonster force-pushed the fam/remove-train-schedules branch from efe3972 to c5ae4e7 Compare January 13, 2026 17:43
@github-actions github-actions bot added the area:integration-tests Work on Integration test, by nature related to different services label Jan 13, 2026
@flomonster flomonster marked this pull request as ready for review January 13, 2026 17:43
@flomonster flomonster requested review from a team as code owners January 13, 2026 17:43
@flomonster flomonster changed the title Cleanup train schedules endpoints models 🧹 Cleanup train schedules endpoints and models 🧹 Jan 13, 2026
Copy link
Contributor

@clarani clarani left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@leovalais leovalais left a comment

Choose a reason for hiding this comment

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

Il y a des news qui rendent le smile ™️

Mostly trusting the CI on this one.

Copy link
Contributor

@shenriotpro shenriotpro left a comment

Choose a reason for hiding this comment

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

LGTM for tests/

@flomonster flomonster force-pushed the fam/remove-train-schedules branch from c5ae4e7 to 0188686 Compare January 14, 2026 15:15
@flomonster flomonster enabled auto-merge January 14, 2026 15:15
@flomonster flomonster added this pull request to the merge queue Jan 14, 2026
Merged via the queue into dev with commit 7ccf194 Jan 14, 2026
30 checks passed
@flomonster flomonster deleted the fam/remove-train-schedules branch January 14, 2026 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:editoast Work on Editoast Service area:front Work on Standard OSRD Interface modules area:integration-tests Work on Integration test, by nature related to different services kind:api-change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants