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

Refactor statemachine_task_status methods into mixin #39

Closed
gmcculloug opened this issue Jun 8, 2017 · 2 comments
Closed

Refactor statemachine_task_status methods into mixin #39

gmcculloug opened this issue Jun 8, 2017 · 2 comments

Comments

@gmcculloug
Copy link
Member

The statemachine_task_status method exists across several service models with only minor differences and should be refactored into a common mixin.

Files to modify:

  • miq_ae_service_automation_task.rb
  • miq_ae_service_miq_host_provision.rb
  • miq_ae_service_miq_provision.rb
  • miq_ae_service_miq_provision_task.rb
  • miq_ae_service_service_reconfigure_task.rb
  • miq_ae_service_service_template_provision_task.rb
  • miq_ae_service_vm_migrate_task.rb
@miq-bot
Copy link
Member

miq-bot commented Dec 11, 2017

This issue has been automatically marked as stale because it has not been updated for at least 6 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions!

@gmcculloug
Copy link
Member Author

These methods will most likely be moved out of the service models and into the backend models.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants