-
Notifications
You must be signed in to change notification settings - Fork 120
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
Move 'dump_root' logic into common method location and include as embedded method #350
Labels
Comments
billfitzgerald0120
added a commit
to billfitzgerald0120/manageiq-content
that referenced
this issue
Jul 16, 2018
This is a new method which can then be included into the other methods using the Embedded method feature. ManageIQ#350
billfitzgerald0120
added a commit
to billfitzgerald0120/manageiq-content
that referenced
this issue
Jul 17, 2018
This is a new method which can then be included into the other methods using the Embedded method feature. Created System/CommonMethods/Utils class. ManageIQ#350
billfitzgerald0120
added a commit
to billfitzgerald0120/manageiq-content
that referenced
this issue
Jul 27, 2018
This is a new method which can then be included into the other methods using the Embedded method feature. Created System/CommonMethods/Utils class. ManageIQ#350 Changed name to LogObject and many other changes ..... Added documentation in the method
billfitzgerald0120
added a commit
to billfitzgerald0120/manageiq-content
that referenced
this issue
Jul 28, 2018
This is a new method which can then be included into the other methods using the Embedded method feature. Created System/CommonMethods/Utils class. ManageIQ#350 Changed name to LogObject and many other changes ..... Added documentation in the method Fixed extra line
billfitzgerald0120
added a commit
to billfitzgerald0120/manageiq-content
that referenced
this issue
Jul 31, 2018
This is a new method which can then be included into the other methods using the Embedded method feature. Created System/CommonMethods/Utils class. ManageIQ#350 Changed name to LogObject and many other changes ..... Added documentation in the method Fixed extra line Fixed rubocop errors in spec
billfitzgerald0120
added a commit
to billfitzgerald0120/manageiq-content
that referenced
this issue
Aug 2, 2018
Added the ability to log all attributes, associations, and tags. Inspectme is a great tool for debugging and now will be available from an embedded method. ManageIQ#350
billfitzgerald0120
added a commit
to billfitzgerald0120/manageiq-content
that referenced
this issue
Aug 3, 2018
Added the ability to log all attributes, associations, and tags. Inspectme is a great tool for debugging and now will be available from an embedded method. ManageIQ#350 Added tests
billfitzgerald0120
added a commit
to billfitzgerald0120/manageiq-content
that referenced
this issue
Aug 13, 2018
Added the ability to log all attributes, associations, and tags. Inspectme is a great tool for debugging and now will be available from an embedded method. ManageIQ#350 Added tests Changed object_prefix to log_prefix as requested.
billfitzgerald0120
added a commit
to billfitzgerald0120/manageiq-content
that referenced
this issue
Aug 13, 2018
Added the ability to log all attributes, associations, and tags. Inspectme is a great tool for debugging and now will be available from an embedded method. ManageIQ#350 Added tests Changed object_prefix to log_prefix as requested. Changed method to obj.respond_to?(:object_class) as requested.
Issue resolved with #386 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The basic "dump_root" logic from the InspectMe method is used in several methods:
The logic should be moved into a new method which can then be included into the other methods using the Embedded method feature. The InspectMe method should also include this logic through the embedded method as well.
Potentially all the methods in InspectMe could be exposed in a utility class for reuse and the InspectMe method today would just become simple calls to those embedded methods.
cc @tinaafitz @mkanoor
The text was updated successfully, but these errors were encountered: