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

feature: API /disruptions with Loki #4165

Merged
merged 1 commit into from
Nov 27, 2023
Merged

feature: API /disruptions with Loki #4165

merged 1 commit into from
Nov 27, 2023

Conversation

datanel
Copy link
Member

@datanel datanel commented Nov 17, 2023

@datanel datanel force-pushed the feature/loki/disruptions branch from 6b93d7f to ac60b0f Compare November 21, 2023 09:25
@datanel datanel requested a review from pbench November 21, 2023 10:46
@pbench pbench requested review from azime and xlqian November 21, 2023 13:02
@datanel datanel force-pushed the feature/loki/disruptions branch from ac60b0f to 01ca102 Compare November 24, 2023 14:08
@@ -40,6 +40,7 @@
from jormungandr.scenarios.utils import places_type, pt_object_type, add_link
from jormungandr.scenarios.utils import build_pagination
from jormungandr.exceptions import UnknownObject
from jormungandr.pt_planners import Loki
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
from jormungandr.pt_planners import Loki

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

@datanel datanel force-pushed the feature/loki/disruptions branch from 01ca102 to f3dcd2b Compare November 27, 2023 10:31
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

50.0% 50.0% Coverage
0.0% 0.0% Duplication

@datanel datanel merged commit 9ec4d3f into dev Nov 27, 2023
@datanel datanel deleted the feature/loki/disruptions branch November 27, 2023 13: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.

None yet

4 participants