Skip to content

editoast, front: rename OperationalPointReference and OperationalPointIdentifier #14479

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
younesschrifi merged 2 commits into from
Jan 6, 2026
Merged

editoast, front: rename OperationalPointReference and OperationalPointIdentifier #14479

younesschrifi merged 2 commits into from
Jan 6, 2026

Conversation

@younesschrifi
Copy link
Contributor

@younesschrifi younesschrifi commented Dec 16, 2025 •
edited
Loading

part of #14107

@github-actions github-actions bot added area:front Work on Standard OSRD Interface modules area:editoast Work on Editoast Service kind:api-change area:railway-manager-interface Work on Railway Manager Interface service labels Dec 16, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 16, 2025

⚠️ API changes

This Pull Request introduces some changes in the API:

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

Changes in Schemas

Dear @younesschrifi, please ensure the following:

@younesschrifi younesschrifi changed the title Yci/operational pointreference v2 p2 editoast, front: rename OperationalPointReference and OperationalPointIdentifier Dec 16, 2025
@younesschrifi younesschrifi force-pushed the yci/operationalPointreferenceV2-p2 branch from 77a4f18 to fe896a8 Compare December 20, 2025 00:41
@younesschrifi younesschrifi force-pushed the yci/operationalPointreferenceV2-p2 branch from fe896a8 to b03b364 Compare January 2, 2026 14:00
@younesschrifi younesschrifi marked this pull request as ready for review January 2, 2026 14:04
@younesschrifi younesschrifi requested review from a team as code owners January 2, 2026 14:04
@younesschrifi younesschrifi requested a review from Khoyo January 2, 2026 14:04
@younesschrifi younesschrifi force-pushed the yci/operationalPointreferenceV2-p2 branch 2 times, most recently from 0cd42e7 to cbf1c31 Compare January 2, 2026 15:52
SharglutDev
SharglutDev reviewed Jan 5, 2026
View reviewed changes
Copy link
Contributor

@SharglutDev SharglutDev left a comment

Choose a reason for hiding this comment

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

Thank for the PR, we're getting close ! :)

@younesschrifi younesschrifi force-pushed the yci/operationalPointreferenceV2-p2 branch 12 times, most recently from 79db3b1 to 5b0dfc8 Compare January 5, 2026 15:56
@younesschrifi younesschrifi force-pushed the yci/operationalPointreferenceV2-p2 branch 3 times, most recently from 143b20f to 1a54fe9 Compare January 5, 2026 16:31
@younesschrifi
front, editoast: rename OperationalPointReference to OperationalPoint…
f996ff6 
…PartReference

Signed-off-by: Youness CHRIFI ALAOUI <youness.chrifi@gmail.com>
@younesschrifi younesschrifi force-pushed the yci/operationalPointreferenceV2-p2 branch from 1a54fe9 to 008f290 Compare January 5, 2026 16:34
@younesschrifi
editoast, front: rename OperationalPointIdentifier to OperationalPoin…
e8bea77 
…tReference

Signed-off-by: Youness CHRIFI ALAOUI <youness.chrifi@gmail.com>
@younesschrifi younesschrifi force-pushed the yci/operationalPointreferenceV2-p2 branch from 008f290 to e8bea77 Compare January 5, 2026 16:50
SharglutDev
SharglutDev approved these changes Jan 6, 2026
View reviewed changes
Copy link
Contributor

@SharglutDev SharglutDev left a comment

Choose a reason for hiding this comment

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

Lgtm !

@younesschrifi younesschrifi added this pull request to the merge queue Jan 6, 2026
aschantraine
aschantraine approved these changes Jan 6, 2026
View reviewed changes
Copy link
Contributor

@aschantraine aschantraine left a comment

Choose a reason for hiding this comment

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

LGTM

Merged via the queue into dev with commit 2ccaf36 Jan 6, 2026
29 checks passed
@younesschrifi younesschrifi deleted the yci/operationalPointreferenceV2-p2 branch January 6, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Khoyo Khoyo Khoyo approved these changes

@flomonster flomonster flomonster approved these changes

@SharglutDev SharglutDev SharglutDev approved these changes

@aschantraine aschantraine aschantraine approved these changes

Assignees

No one assigned

Labels

area:editoast Work on Editoast Service area:front Work on Standard OSRD Interface modules area:railway-manager-interface Work on Railway Manager Interface service kind:api-change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@younesschrifi @Khoyo @flomonster @SharglutDev @aschantraine