Skip to content

feat: add link from risk of confusion to identify operator - #1225

Open
therobrob wants to merge 5 commits into
mainfrom
1079-add-link-from-risk-of-confusion-to-identify-operator
Open

therobrob wants to merge 5 commits into
mainfrom
1079-add-link-from-risk-of-confusion-to-identify-operator

Conversation

@therobrob

Copy link
Copy Markdown
Member

No description provided.

@therobrob therobrob linked an issue Oct 4, 2026 that may be closed by this pull request
@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown

Hi there! 👋

Thank you for your contribution to the FIP Guide! 🚀
We appreciate your effort in making rail staff travel information more accessible. 🚄

Checklist before merging:

  • Added a description to the Pull Request
  • Checked the License of new pictures (non-commercial use without attribution)
  • Modified content in English
  • Modified content in German
  • Modified content in French

@netlify

netlify Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for fipguide ready!

Name Link
🔨 Latest commit 9108927
🔍 Latest deploy log https://app.netlify.com/projects/fipguide/deploys/6ac6991c06f89a00083095a7
😎 Deploy Preview https://deploy-preview-1225--fipguide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@therobrob therobrob added content Improvements or additions to the content (countries, news, operators, general content) technical Technical issues, e.g. related to Hugo, HTML, CSS, deployment, etc. labels Oct 4, 2026

@lenderom lenderom left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank's Robert, I really like this one.

At first I tought, maybe we could get the country from the site context. But then I remembered that county <-> operator isn't a 1:n relation but a n:m relation. So it makes sense to specify this at the shortcode. Furthermore we can catch some edge cases like GYSEV with this approach.

@MoritzWeber0 MoritzWeber0 left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the implementation and the cool idea 💯

Comment thread layouts/_partials/highlight.html
Comment thread content/operator/zssk/index.fr.md Outdated
…tify-operator

# Conflicts:
#	content/operator/sncf/index.de.md
#	content/operator/sncf/index.en.md
#	content/operator/sncf/index.fr.md
@therobrob
therobrob requested a review from MoritzWeber0 October 7, 2026 19:10

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content Improvements or additions to the content (countries, news, operators, general content) technical Technical issues, e.g. related to Hugo, HTML, CSS, deployment, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add link from risk of confusion to identify-operator

3 participants