Skip to content

Slight adjustment to rule_priority in the Fares Implementation Guide - #649

Merged
skalexch merged 2 commits into
mainfrom
update-implementation-guide
Jan 19, 2026
Merged

Slight adjustment to rule_priority in the Fares Implementation Guide#649
skalexch merged 2 commits into
mainfrom
update-implementation-guide

Conversation

@skalexch

Copy link
Copy Markdown
Contributor

No description provided.

@skalexch

Copy link
Copy Markdown
Contributor Author

After having reviewed the guide, we do not need to update anything major except what's in this PR. We do not say anything incorrect or dangerous in rule_priority.
We also kept things simple with transfers so the semantics clarification does not induce any change in the guide on our side.
Tagging @abigailmbd for documentation review and @Sergiodero to make sure what I added is not too complex.

Next step in the guide is to draft new content related to new and upcoming proposals:

  • cEMV support
  • Area Sets + Leg Join
    These will appear in their own PRs.

@skalexch

Copy link
Copy Markdown
Contributor Author

@Sergiodero should I change "predicates" to "conditions"?

@abigailmbd abigailmbd left a comment

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.

The changes look good, though I would agree that "conditions" is clearer here than "predicates."

@Sergiodero Sergiodero left a comment

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.

100% agree with using "conditions" over "predicates", other than that LGTM!

@Sergiodero

Copy link
Copy Markdown
Contributor

Also, just flagging that we currently do not have translations for this guide. We should add this to the backlog once the migration to the resources section is completed.

@skalexch
skalexch merged commit 805be7d into main Jan 19, 2026
@skalexch
skalexch deleted the update-implementation-guide branch January 19, 2026 18:53
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