Skip to content

feat: Add taxation calculator for DB employees - #814

Merged
lenderom merged 49 commits into
mainfrom
feature/taxation
Sep 24, 2026
Merged

lenderom merged 49 commits into
mainfrom
feature/taxation

Conversation

@lenderom

@lenderom lenderom commented Apr 6, 2026

Copy link
Copy Markdown
Member

No description provided.

@netlify

netlify Bot commented Apr 6, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for fipguide ready!

Name Link
🔨 Latest commit c558ff0
🔍 Latest deploy log https://app.netlify.com/projects/fipguide/deploys/6ab4cd8ee7c9ff0008847284
😎 Deploy Preview https://deploy-preview-814--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.

@github-actions

github-actions Bot commented Apr 6, 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

@lenderom
lenderom force-pushed the feature/taxation branch 3 times, most recently from 0082495 to fd56ed3 Compare April 6, 2026 20:30
@lenderom
lenderom marked this pull request as draft April 7, 2026 11:43
@therobrob therobrob added enhancement New feature or request technical Technical issues, e.g. related to Hugo, HTML, CSS, deployment, etc. labels Apr 11, 2026
@therobrob therobrob linked an issue Apr 12, 2026 that may be closed by this pull request
@MoritzWeber0 MoritzWeber0 changed the title Feature/taxation feat: Add taxation calculator for DB employees Apr 16, 2026
@lenderom

Copy link
Copy Markdown
Member Author

Wir entwickeln das Feature im DB internal weiter zu Ende. Einbindung z.B. per iFrame oder Link. Zugriff nur aus Intranet möglich.

@lenderom

lenderom commented Aug 5, 2026 •

Copy link
Copy Markdown
Member Author
  • hint for allow network acces
  • warum so viele issuer
  • larger hint, this is not official
  • tests and document order of optimazation

@MoritzWeber0

MoritzWeber0 commented Aug 19, 2026 •

Copy link
Copy Markdown
Member
  • Taxation as part of the FIP application page
  • Taxation calculator should be on separate page
  • Check Permissions API
  • Resolve conflicts
  • Technical cleanup

@lenderom

lenderom commented Sep 11, 2026 •

Copy link
Copy Markdown
Member Author
  • Beschriftung der Sonstigen Sachbezüge anpassen, damit sichtbar wird, dass eine Zahl eingegeben werden soll.
  • Warnungsicon konsistent nutzen (mit @therobrob abstimmen. Ich hab jetzt global mal das warning icon statt campaing gesetzt und auch inofficial angepasst. Idee war dass warning vielleicht auch mehr im rechner passt statt campaign und wir das dann einheitlich haben. Aber ich kann auch gerne campaign im Rechner nutzen. Das sah nur ein bisschen merkwürdig aus.)
  • Schloss ausblenden in den nicht zugewiesenen Monaten
  • Buttons mobil rechtsbündig und immer quadratisch
  • Border radius Planungsmodus anpassen
  • Zahlen immer schwarz
  • Summenicon entfernen
  • Planungsmodus nicht nur hover beim reinziehen, sondern immer
  • Besonderheiten einzelne Konzernunternehmen inhaltlich prüfen
  • News Article
  • Andere Sprachen hinzufügen
  • Abstand im Summenfeld nicht konsistent
  • Typische Abkürzungen in die Namen einbringen, dass z.B. auch GB funktioniert in der Suche und Konsistenz zu Operatorüberschriften
  • HZP und ZRSM Logo fehlen, weil die Artikel noch nicht da sind (würde ich erstmal als okay betrachten und keine Sonderlogik bauen)
  • Discuss if we need autoscroll on mobile and/ord Desktop. iOS it doesn't work well on iOS and idk on Android.
  • Überschriften der Highlights durchsprechen. Die sind global aktuell gesetzt
  • Netzcard hinzufügen
  • Add Gesamtübersicht
  • Netzcard fällt auch in diese Kategorie
  • Allgemeiner Hinweis beim Laden statt Abfragen der API
  • Kleinerer Hinweis keine Freifahrten für Angehörige
  • Anleitung rausnehmen
  • Ins Menü aufnehmen
  • Minifinzieren in Pipeline
  • Wird geladen Icon

@lenderom
lenderom marked this pull request as ready for review September 20, 2026 16:34

@therobrob therobrob 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.

Hi @lenderom, ich bin den PR jetzt mal komplett durchgegangen und habe ein paar Kommentare da gelassen. Schau gern mal bei Gelegenheit durch :) Freue mich darauf, wenn wir das Feature live nehmen, danke für deine Arbeit! ^~^

Comment thread content/application/db/index.de.md Outdated
Comment thread content/application/db/index.de.md Outdated
Comment thread content/news/14/index.de.md
Comment thread content/news/14/index.de.md Outdated
Comment thread content/news/14/index.de.md
Comment thread layouts/_partials/train-category.html
Comment thread layouts/_shortcodes/taxation-embed.html Outdated
Comment thread layouts/_shortcodes/taxation-embed.html Outdated
Comment thread layouts/_shortcodes/taxation-embed.html Outdated
Comment thread layouts/_shortcodes/taxation-embed.html
Comment thread layouts/_partials/highlight.html Outdated
@therobrob

Copy link
Copy Markdown
Member

Wollen wir, dass der Steuerrechner in den Suchindizes auftaucht? Falls nicht, sollten wir die robots-advices auf "noindex,nofollow" stellen.

@lenderom
lenderom merged commit 0fafe02 into main Sep 24, 2026
7 checks passed
@lenderom
lenderom deleted the feature/taxation branch September 24, 2026 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request 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.

calculation tool "Geldwerter Vorteil"

3 participants