fix(renfe): Add image and clarify class options #160
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MoritzWeber0
added
content
✅ Deploy Preview for fipguide ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
MoritzWeber0
force-pushed
the
fix/renfe-classes
branch
from
cffcb88 to
b0fc50a
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR renames the Elige tariff entries for clarity, adds external links in the class definitions, and updates the DB Travel Center instructions to reflect the new class codes.
- Standardizes the FIP Global Fare list items to use “Elige Estándar”
- Expands the “Class Categories” section with more detailed descriptions and links
- Clarifies on-site booking rules and adds note on Premium class availability
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| content/operator/renfe/index.en.md | Updated fare bullets, expanded class definitions with links, and clarified DB Travel Center instructions |
| content/operator/renfe/index.de.md | Mirrored English updates in German, including heading rename and class details with links |
Comments suppressed due to low confidence (2)
content/operator/renfe/index.en.md:124
- These items were previously rendered as a markdown list but no longer start with
-. Consider restoring the leading hyphens (-) so they render as bullets consistently.
**Elige Standard / Turista**: Comparable to 2nd class. [More Information](https://www.renfe.com/es/en/travel/prepare-your-trip/billetes-ave-y-largadistancia/elige) \
content/operator/renfe/index.de.md:125
- After changing the list to bold paragraphs, the leading
-was removed and the items no longer render as a bullet list. Re-add the hyphen to maintain proper markdown list formatting.
**Elige Standard / Turista**: Vergleichbar mit der 2. Klasse. [Mehr Informationen](https://www.renfe.com/es/en/travel/prepare-your-trip/billetes-ave-y-largadistancia/elige) \
lenderom
reviewed
May 28, 2025
MoritzWeber0
force-pushed
the
fix/renfe-classes
branch
from
dde0ebc to
e4dacd2
Compare
lenderom
approved these changes
May 28, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The tariff and class options were not clean, esp. the reference to the "Preferente" class.
Once #157 is merged, I'll rebase the PR.