Skip to content

GTFS-RT multi-coach occupancy - Merge Lyft proposal with TfNSW and LIRR proposals#58

Closed
barbeau wants to merge 7 commits intomasterfrom
lfontolliet-multi-car-crowding
Closed

GTFS-RT multi-coach occupancy - Merge Lyft proposal with TfNSW and LIRR proposals#58
barbeau wants to merge 7 commits intomasterfrom
lfontolliet-multi-car-crowding

Conversation

@barbeau
Copy link
Member

@barbeau barbeau commented Aug 5, 2020

Main changes to Lyft proposal:

  • Change terminology to "Coach" and "Status"
  • Add sequence number to coaches
  • In case of unknown values, provide the sequence number and do NOT provide the unknown field
  • Add reference documentation (in addition to .proto changes)
  • Add "experimental" tags

lfontolliet and others added 5 commits July 27, 2020 11:28
Primary changes:
* Change terminology to "Coach" and "Status"
* Add sequence number to coaches
* In case of unknown values, provide the sequence number and do NOT provide the unknown field
@CLAassistant
Copy link

CLAassistant commented Aug 5, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ timMillet
✅ barbeau
❌ lfontolliet
You have signed the CLA already but the status is still pending? Let us recheck it.

@barbeau
Copy link
Member Author

barbeau commented Aug 18, 2020

Closing, these changes are now reflected in google#237.

@barbeau barbeau closed this Aug 18, 2020
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.

4 participants