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 towhat to resource_type #4890

Closed
wants to merge 2 commits into from
Closed

Rename towhat to resource_type #4890

wants to merge 2 commits into from

Conversation

juliancheal
Copy link
Member

@juliancheal juliancheal commented Nov 7, 2018

Fixes deprecation warnings of towhat.

This PR needs to be merged in order see issue for ordering ManageIQ/manageiq#18222

@martinpovolny
Copy link
Member

Looks good.

@himdel
Copy link
Contributor

himdel commented May 10, 2019

Looks like this depends on ManageIQ/manageiq-schema#314

Julian Cheal added 2 commits October 18, 2019 13:45
Fixes deprecation warnings of towhat.

Links to this PR ManageIQ/manageiq#18165
and this PR ManageIQ/manageiq-schema#301
@miq-bot
Copy link
Member

miq-bot commented Oct 18, 2019

Checked commits https://github.com/juliancheal/manageiq-ui-classic/compare/18a830c161bcec4870a2f5ff4c4d839055f8d976~...df7c6cea2d93ca2431c5e49100f5c53c4e8f860a with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
19 files checked, 5 offenses detected

app/controllers/miq_policy_controller/conditions.rb

app/helpers/application_helper/toolbar/condition_center.rb

spec/presenters/tree_builder_protect_spec.rb

@juliancheal
Copy link
Member Author

@miq-bot remove_label unmergeable

@mfeifer
Copy link
Contributor

mfeifer commented Mar 6, 2020

@bdunne

@bdunne
Copy link
Member

bdunne commented Mar 6, 2020

cc @jrafanie with your rebase effort...

@jrafanie
Copy link
Member

jrafanie commented Mar 6, 2020

This can be closed, I have cherry-picked/rebased the changes here into a replacement PR #6742 since I didn't have push rights on the original fork.

@himdel himdel closed this Apr 6, 2020
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.

9 participants