-
-
Notifications
You must be signed in to change notification settings - Fork 19
fix: update relationship of old news #576
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Hi there! 👋 Thank you for your contribution to the FIP Guide! 🚀 Checklist before merging:
|
✅ Deploy Preview for fipguide ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
MoritzWeber0
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Danke! Im News Artikel von LTG steht, dass wir LTG in Kürze ergänzen. Das könnten wir noch entfernen oder anpassen.
Inwiefern? Die News hat Datum, zu welchem die Information galt. Wenn wir die Info anpassen, suggerieren wir, dass diese Information zu dem Zeitpunkt bestand. Oder nicht? |
Würde mal behaupten, dass die meisten Leute nicht auf das Datum schauen, daher könnte das verwirrend wirken und der Eindruck entstehen, dass wir das bisher nicht auf der Website haben, z. B. wenn man direkt über Google auf den Newsartikel kommt. Ich finde den Vorschlag gut, wenn wir es explizit als Update kennzeichnen, so wie bei der Eurostar Preiserhöhung: https://www.fipguide.org/de/news/4/ |
|
Sorry ich hab eure Diskussion hier jetzt nicht verfolgt und das einfach nach bestem Wissen und gewissen angepasst. Ich weiß nicht, ob wir hier eine "Update-Kennzeichnung" brauchen. Es hat sich ja nicht direkt eine Information verändert. Auch wenn ich da sonst voll bei @therobrob und das sehr wichtig finde. In diesem Fall finde ich es vernachlässigbar und es ist den Leuten vermutlich eh egal. Wenn ihr es doch haben wollt, könnt ihr es aber auch gerne anpassen. In jedem Fall haben wir über Github ja die Bearbeitungshistorie und damit wahrscheinlich mehr als 99% der Websites :D |
| draft: false | ||
| author: "Lennart" | ||
| title: "Neues FIP Mitglied: LTG Link (Litauen)" | ||
| title: "Update: Neues FIP Mitglied: LTG Link (Litauen)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das "Update:" würde ich eher weglassen. Der Artikel bezieht sich ja nicht auf ein Update eines früheren Artikels, sondern den Inhalt des Artikels selbst. Der Vermerk innerhalb des Artikels reicht meines Erachtens :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sollen wir das bei der Eurostar News dann auch wegmachen? Weil da haben wir das so gemacht. :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oder meinst du, da ist es dann angebracht, weil es eine echte inhaltlich Änderung gab? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Würde das Update im Titel bei Eurostar drinlassen, hier aber rausnehmen, da es sich hier nicht um eine inhaltliche Änderung handelt, sondern lediglich redaktionell ist.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mhm gute Frage. Das "Update" bezieht sich ja nicht auf den Titel, sondern auf den Status des Artikels. Es wäre also eher etwas wie "News-Titel (updated)". Bin unschlüssig :'D
Update the related countries and operators of the existing news