Skip to content
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

Add KoliBriDevHelper for theme patching and custom translations #7146

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

deleonio
Copy link
Contributor

@deleonio deleonio commented Dec 4, 2024

Refs: #7144

The A11y and PO reviews will only take place after all other DoD steps have been completed by the Developer:

  • Meaningful pull request title for the release notes
  • Pull request is linked to an issue and all changes relate to the issue
  • Tests to protect this code implemented (if applicable)
  • Manual test performed successfully (if applicable)
  • Documentation or migration has been updated (if applicable)

Copy link
Contributor

github-actions bot commented Dec 4, 2024

@deleonio deleonio enabled auto-merge December 4, 2024 16:05
@deleonio deleonio requested a review from sdvg December 4, 2024 16:05
@deleonio deleonio added the v3 Here are issues that need to be resolved for version 3. label Dec 4, 2024
@deleonio deleonio linked an issue Dec 4, 2024 that may be closed by this pull request
Copy link
Member

@sdvg sdvg left a comment

Choose a reason for hiding this comment

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

Ich finde es gut, wenn es Beispiele gibt, die zeigen, wie die Overrides funktionieren.

Ich frage mich nur, ob diese in der Sample App so gut aufgehoben sind. Auf der einen Seite ist die Sample App natürlich für solche Dinge da, auf der anderen Seite kann es aber auch irritieren, wenn man an den betreffenden Stellen arbeitet und erst mal nicht weiß, warum die Styles überschrieben oder eine alternative Übersetzung gezeigt werden.

Vorschlag:
Könnten wir diese Modifikationen hinter einem Flag verstecken, analog dazu wie wir es hier für den Tag Name Transformer gemacht haben?

Damit das Beispiel so richtig nützlich wird, könnten wir dann auch noch einen kleinen E2E-Test bauen, der das Flag aktiviert und die Overrides prüft.

@deleonio
Copy link
Contributor Author

deleonio commented Dec 4, 2024

Ich möchte alle Code-Samples jeder Art darin zusammenhalten. Das mit dem Flag finde ich gut. Schaue ich mir morgen früh gleich mal an.

@deleonio deleonio disabled auto-merge December 5, 2024 18:32
@deleonio deleonio merged commit 4e9d023 into develop Dec 5, 2024
8 checks passed
@deleonio deleonio deleted the sample/7144-v3 branch December 5, 2024 18:32
@github-actions github-actions bot locked and limited conversation to collaborators Dec 5, 2024
@deleonio deleonio linked an issue Dec 11, 2024 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
v3 Here are issues that need to be resolved for version 3.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add-sample-for-translations-and-patch-theme Sprachspezifische-Texte in KoliBri - Beispiel aufbauen
2 participants