Skip to content

Conversation

@therobrob
Copy link
Member

@therobrob therobrob commented Jan 18, 2026

  • changed traincategory expander in gb, bsb and cfl

@github-actions
Copy link

github-actions bot commented Jan 18, 2026

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
Copy link

netlify bot commented Jan 18, 2026

Deploy Preview for fipguide ready!

Name Link
🔨 Latest commit dcbc06b
🔍 Latest deploy log https://app.netlify.com/projects/fipguide/deploys/696cdcc3c9bc8700084677c9
😎 Deploy Preview https://deploy-preview-613--fipguide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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) operator::gb operator::cfl labels Jan 18, 2026
@therobrob therobrob requested a review from lenderom January 18, 2026 11:59
@therobrob
Copy link
Member Author

therobrob commented Jan 18, 2026

@MoritzWeber0 what does "missing" mean in this context? pre-commit didn’t mark any errors?

@MoritzWeber0
Copy link
Member

@MoritzWeber0 what does "missing" mean in this context? pre-commit didn’t mark any errors?

prettier can't auto-format it unfortunately (I was thinking about a custom prettier rule, but didn't look into it yet), but it improves readability in my opinion. That's why I added the indentation to the archetype and it's used by all other occurrences. For the booking expanders we did the same, so for consistency reasons I added them here as well :)

Copy link
Member

@MoritzWeber0 MoritzWeber0 left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM :)

@MoritzWeber0 MoritzWeber0 merged commit ca11dcd into main Jan 18, 2026
7 checks passed
@MoritzWeber0 MoritzWeber0 deleted the feat/change-traincategory-expander branch January 18, 2026 13:26
@therobrob
Copy link
Member Author

Zwei Fragen sind mir bei der Bearbeitung noch gekommen:

  • teilweise übersetzen wir die Titel der Zugkategorien, teilweise sind. Wollen wir das einheitlich machen? Ich persönlich finde, dass wir den Namen aus der Landessprache bzw. den Eigennamen nutzen sollten, außer es ist ein allgemeiner Name wie "Standseilbahn" oder "Bus"
  • sollten die IDs der train-category-Expander einheitlich in englisch sein? Haben wir eine Namenskonvention?

@MoritzWeber0
Copy link
Member

Zwei Fragen sind mir bei der Bearbeitung noch gekommen:

  • teilweise übersetzen wir die Titel der Zugkategorien, teilweise sind. Wollen wir das einheitlich machen? Ich persönlich finde, dass wir den Namen aus der Landessprache bzw. den Eigennamen nutzen sollten, außer es ist ein allgemeiner Name wie "Standseilbahn" oder "Bus"

Das finde ich eine gute Grundlage. Teilweise haben die Operator aber auch verschiedene, offizielle Namen in den Sprachen, dann können wir die auch nehmen. Beispiel ist dort in der Schweiz die Rhätische Bahn, die auf Englisch tatsächlich "Rhaetian Railway" heißt.

  • sollten die IDs der train-category-Expander einheitlich in englisch sein? Haben wir eine Namenskonvention?

Die einzige Konvention, die wir derzeit haben, steht im archetype: "Kleingeschriebene, mit Bindestrichen getrennte Abkürzung der Zugkategorie (z.B. "ice", "tgv-inoui")."

Im Optimalfall ist es unabhängig von der Sprache und möglichst stabil, da die ID in Links verwendet wird, die auch noch Monate später funktionieren sollen. Und es wäre gut, wenn sie einheitlich über alle Sprachen hinweg sind, das macht es beim Übersetzen der Links einfacher.

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) operator::cfl operator::gb

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants