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

[tech] Add offers for NeTEx France #544

Merged
merged 8 commits into from
Feb 26, 2020

Conversation

woshilapin
Copy link
Contributor

No description provided.

@woshilapin woshilapin force-pushed the netex-france-offre-skeleton branch 3 times, most recently from 0b77549 to c1adf7d Compare February 12, 2020 16:23
Comment on lines 21 to 27
enum ObjectType {
_Route,
_ServiceJourneyPattern,
_ScheduledStopPoint,
_PassengerStopAssignment,
_ServiceJourney,
}

This comment was marked as outdated.

@woshilapin woshilapin force-pushed the netex-france-offre-skeleton branch 5 times, most recently from 27bd821 to 838229f Compare February 18, 2020 09:15
@woshilapin woshilapin changed the title [tech] Create skeleton for offers in NeTEx France [tech] Add offers for NeTEx France Feb 18, 2020
@woshilapin

This comment has been minimized.

@woshilapin woshilapin force-pushed the netex-france-offre-skeleton branch 4 times, most recently from cbde9d0 to 0bca2e6 Compare February 24, 2020 13:35
ArnaudOggy
ArnaudOggy previously approved these changes Feb 25, 2020
Self::generate_stop_sequence_id(
&vehicle_journey_id,
stop_time.sequence,
ObjectType::ScheduledStopPoint,

This comment was marked as outdated.

This comment was marked as outdated.

@woshilapin woshilapin force-pushed the netex-france-offre-skeleton branch from 0bca2e6 to 1788ac2 Compare February 25, 2020 12:59
@mergify mergify bot dismissed ArnaudOggy’s stale review February 25, 2020 12:59

Pull request has been modified.

@woshilapin woshilapin force-pushed the netex-france-offre-skeleton branch 2 times, most recently from d9f9a9f to 28b64c4 Compare February 25, 2020 14:41
ArnaudOggy
ArnaudOggy previously approved these changes Feb 25, 2020
@@ -314,6 +382,36 @@ impl<'a> OfferExporter<'a> {
.build()
}

fn generate_day_type_ref(vehicle_journey_id: &'a str) -> Element {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
fn generate_day_type_ref(vehicle_journey_id: &'a str) -> Element {
fn generate_day_type_ref(service_id: &'a str) -> Element {

This comment was marked as outdated.

@woshilapin woshilapin force-pushed the netex-france-offre-skeleton branch from 28b64c4 to c7ced1b Compare February 25, 2020 15:15
@mergify mergify bot dismissed ArnaudOggy’s stale review February 25, 2020 15:15

Pull request has been modified.

@woshilapin woshilapin force-pushed the netex-france-offre-skeleton branch from ea039be to dc69876 Compare February 26, 2020 09:52
@woshilapin woshilapin force-pushed the netex-france-offre-skeleton branch from 7213dc8 to 4712376 Compare February 26, 2020 17:04
@ArnaudOggy ArnaudOggy merged commit cd83295 into hove-io:master Feb 26, 2020
@woshilapin woshilapin deleted the netex-france-offre-skeleton branch February 26, 2020 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants