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

Custom branding v2 #1335

Closed
wants to merge 1 commit into from
Closed

Conversation

Ironem
Copy link

@Ironem Ironem commented Mar 9, 2022

Signed-off-by: Jiawei YANG [email protected]

Description

Add the feature describe in the feature request #1250.

Issues Resolved

#1250 Custom branding v2

Check List

  • New functionality includes testing.
    • All tests pass
      • yarn test:jest
      • yarn test:jest_integration
      • yarn test:ftr
  • New functionality has been documented.
  • Commits are signed per the DCO using --signoff

@ananzh ananzh requested review from kavilla, tmarkley and ananzh March 15, 2022 21:41
@tmarkley
Copy link
Contributor

Hey @Ironem thanks for the PR! Can we continue the discussion about this approach over in #1250? I think there are still a couple things we need to ahem iron out.

@joshuarrrr
Copy link
Member

Hi @Ironem - thanks again for opening this PR. As discussed in #2044 (comment) and #1250 (comment), the maintainer team would like to avoid introducing any further configuration settings, because each one becomes a backwards compatibility maintenance issue.

Particularly for customizing strings, our preferred approach is to use the i18n plugin which already provides tooling for customizing or overriding any default strings. If there are strings that are not correctly configured for the plugin, feel free to raise issues or PRs to add the i18n wrappers.

Sorry for the delay and uncertainty, we just want to make sure we don't commit to features we can't support long-term.

@joshuarrrr joshuarrrr closed this Sep 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants