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

Rename 'Ansible Tower' to 'Ansible Automation Platform' #318

Conversation

nasark
Copy link
Member

@nasark nasark commented Dec 11, 2024

@miq-bot assign @agrare
@miq-bot add_reviewer @agrare

@@ -15,8 +15,8 @@ def run_with_miq_job(options, userid = nil)
end

def self.display_name(number = 1)
n_('Job Template (Ansible Tower)', 'Job Templates (Ansible Tower)', number)
n_('Job Template (Ansible Automation Platform)', 'Job Templates (Ansible Automation Platform)', number)
Copy link
Member

@agrare agrare Dec 11, 2024

Choose a reason for hiding this comment

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

For a follow-up but I wonder how many of these can be dropped if we just did

Suggested change
n_('Job Template (Ansible Automation Platform)', 'Job Templates (Ansible Automation Platform)', number)
n_("Job Template (#{module_parent.description})", "Job Templates (#{module_parent.description})", number)

in the base AWX class

Copy link
Member

Choose a reason for hiding this comment

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

Because there's the n_, you can't use interpolation, but you could do:

Suggested change
n_('Job Template (Ansible Automation Platform)', 'Job Templates (Ansible Automation Platform)', number)
n_('Job Template (%{provider_description}', 'Job Templates (%{provider_description)', number % {:provider_description => module_parent.description}

@agrare agrare merged commit ab3869b into ManageIQ:master Dec 11, 2024
4 checks passed
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.

3 participants