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

BZ#1525880-Ensure service tag filters flagged for translation #1343

Merged

Conversation

AllenBW
Copy link
Member

@AllenBW AllenBW commented Dec 14, 2017

closes https://bugzilla.redhat.com/show_bug.cgi?id=1525880

The "results" and "selected text sadly aren't within our control, provided by ang-pf, looking into possible options for translating...

@AllenBW AllenBW requested a review from himdel as a code owner December 14, 2017 15:54
@AllenBW AllenBW added this to the Sprint 76 Ending Jan 1, 2018 milestone Dec 14, 2017
@AllenBW AllenBW changed the title Ensure service tag filters flagged for translation BZ#1525880-Ensure service tag filters flagged for translation Dec 14, 2017
@miq-bot
Copy link
Member

miq-bot commented Dec 14, 2017

Checked commit AllenBW@0894bf3 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

Copy link
Contributor

@himdel himdel left a comment

Choose a reason for hiding this comment

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

LGTM, adds i18n :)

@himdel himdel merged commit 27c851f into ManageIQ:master Dec 20, 2017
@himdel
Copy link
Contributor

himdel commented Dec 20, 2017

..provided by ang-pf, looking into possible options for translating...

AFAIK the only option is to create a patternfly PR to allow passing in those strings to override their default ones.

(Which sometimes means we have to temporrarily copy the component out of PF to be able to backport.)

simaishi pushed a commit that referenced this pull request Jan 3, 2018
…d-filter-text

BZ#1525880-Ensure service tag filters flagged for translation
(cherry picked from commit 27c851f)
@simaishi
Copy link
Contributor

simaishi commented Jan 3, 2018

Gaprindashvili backport details:

$ git log -1
commit 8bc6865f73cad7784f8bc49ce99742b8bbdb8793
Author: Martin Hradil <[email protected]>
Date:   Wed Dec 20 20:24:24 2017 +0100

    Merge pull request #1343 from AllenBW/bug/master/#1525880-untranslated-filter-text
    
    BZ#1525880-Ensure service tag filters flagged for translation
    (cherry picked from commit 27c851fbe1b75504503b498d93a8bd4dff7e5776)

@AllenBW AllenBW deleted the bug/master/#1525880-untranslated-filter-text branch January 8, 2018 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants