Skip to content

feat: added travel report from zugpost#904

Merged
therobrob merged 5 commits intomainfrom
feat/cp-zugpost
May 2, 2026
Merged

feat: added travel report from zugpost#904
therobrob merged 5 commits intomainfrom
feat/cp-zugpost

Conversation

@therobrob
Copy link
Copy Markdown
Member

Added travel report from Zugpost to german CP page

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 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
Copy Markdown

netlify Bot commented May 1, 2026

Deploy Preview for fipguide ready!

Name Link
🔨 Latest commit 6475fdd
🔍 Latest deploy log https://app.netlify.com/projects/fipguide/deploys/69f5e31f82d0ff00088591fd
😎 Deploy Preview https://deploy-preview-904--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.

MoritzWeber0
MoritzWeber0 previously approved these changes May 1, 2026
Copy link
Copy Markdown
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.

Schöner Bericht, danke fürs Ergänzen :)

@lenderom
Copy link
Copy Markdown
Member

lenderom commented May 2, 2026

Adding this great trip report is a really good idea. I appreciate it :D
However what do you think about also adding it also in english and german. I think the selected language should not be relevant for the shown content. However we could add something like (only in german) behind the link. But with modern browser translation this shouldn't be a problem for most users. :)

@MoritzWeber0
Copy link
Copy Markdown
Member

Adding this great trip report is a really good idea. I appreciate it :D
However what do you think about also adding it also in english and german. I think the selected language should not be relevant for the shown content. However we could add something like (only in german) behind the link. But with modern browser translation this shouldn't be a problem for most users. :)

For Euskotren we didn't add it to the English and French pages, so if we change it, we should also change it there :)

@lenderom
Copy link
Copy Markdown
Member

lenderom commented May 2, 2026

Yeah that's fine for me. I will change it on the Euskotren site and will wait what @therobrob thinks about it :)

@therobrob
Copy link
Copy Markdown
Member Author

Yeah that's fine for me. I will change it on the Euskotren site and will wait what @therobrob thinks about it :)

That’s okay :) there is an third link on the German renfe page.

@lenderom
Copy link
Copy Markdown
Member

lenderom commented May 2, 2026

I updated this. I also added (nur auf Deutsch) on the german page to keep the translation consistent. Even it's not 100% perfect. But I guess it's okay.

@therobrob therobrob merged commit 5d4c16e into main May 2, 2026
7 checks passed
@therobrob therobrob deleted the feat/cp-zugpost branch May 2, 2026 11:43
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