Skip to content

MTC Tweak route attribute editor docs #803

Merged
binh-dam-ibigroup merged 1 commit intomtc-docsfrom
mtc-route-attr-doc-fixes-2
Jun 13, 2022
Merged

MTC Tweak route attribute editor docs #803
binh-dam-ibigroup merged 1 commit intomtc-docsfrom
mtc-route-attr-doc-fixes-2

Conversation

@binh-dam-ibigroup
Copy link
Copy Markdown
Contributor

@binh-dam-ibigroup binh-dam-ibigroup commented Jun 9, 2022

Checklist

  • Appropriate branch selected => This is a MTC documentation-only update against the mtc-docs branch.
  • [na] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
  • [na] The description lists all applicable issues this PR seeks to resolve
  • [na] The description lists any configuration setting(s) that differ from the default settings
  • All tests and CI builds passing

Description

Documentation tweaks requested by client.

Copy link
Copy Markdown
Contributor

@miles-grant-ibigroup miles-grant-ibigroup left a comment

Choose a reason for hiding this comment

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

Looks good! One small thing I'll leave up to the second reviewer.


#### Using `route_attributes.txt` from another feed version

**Important!** To inject GTFS+ edits done to another version, **the edits need to be published first to a new GTFS version** (see [Publishing a new feed version](#publishing-a-new-feed-version) above). In the last step below, select that published version as the source for the route attributes to reuse.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm a bit confused by this, but maybe if I used the gtfs+ module regularly it would make more sense. Will leave it up to the second reviewer!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That bit of text was requested by the client to remind their users about that.

@binh-dam-ibigroup
Copy link
Copy Markdown
Contributor Author

Going ahead and merge with one approval as discussed.

@binh-dam-ibigroup binh-dam-ibigroup merged commit bc990f7 into mtc-docs Jun 13, 2022
@binh-dam-ibigroup binh-dam-ibigroup deleted the mtc-route-attr-doc-fixes-2 branch June 13, 2022 18:06
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.

3 participants