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

Replace "Satellite" with "Foreman" in user-facing strings. #897

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

Thorben-D
Copy link
Contributor

This PR provides a starting point for un-branding this plugin.

Only user-facing strings are un-branded right now. Un-branding environment variables or hash-keys would either break existing usages or create a bit of a mess. I am not sure that would be worth the trouble...

Using foreman_theme_satellite in conjunction with this plugin re-applies the branding.

Copy link
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

Looks fine, can you fix the failing snapshot JS test?

@Thorben-D
Copy link
Contributor Author

Tests should be working now!

@ShimShtein
Copy link
Member

Let's wait for the branching, and then push the PR. We are after the strings were sent to translations, so changing the templates will result in breakage.

@sbernhard
Copy link
Contributor

Is the branching already done @ShimShtein ?
maybe have a look at #901, too

@chris1984
Copy link
Member

Is the branching already done @ShimShtein ? maybe have a look at #901, too

Branching will happen after we go GA downstream. There is discussion still about the branch name for the next version too.

@Thorben-D Thorben-D changed the base branch from v10 to develop October 1, 2024 09:36
@Thorben-D
Copy link
Contributor Author

Rebased to "develop"

Copy link
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

Looks good, Thanks @Thorben-D

@chris1984 chris1984 merged commit f8f2f49 into theforeman:develop Oct 15, 2024
21 checks passed
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.

4 participants