-
Notifications
You must be signed in to change notification settings - Fork 115
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
[6.12.z] - remove the all nailgun imports from UI folder #14433
[6.12.z] - remove the all nailgun imports from UI folder #14433
Conversation
trigger: test-robottelo |
PRT Result
|
4304b10
to
4624e6a
Compare
trigger: test-robottelo |
@omkarkhatavkar Could you rebase and update this PR? |
4624e6a
to
6bc35a7
Compare
@Gauravtalreja1 @jyejare Rebase Done ! |
@omkarkhatavkar Could you rebase this PR again? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Rebase & conflicts resolve needed.
6.12.z is now absolute hence closing this |
Problem Statement
We are using the direct imports from the nailgun rather than directly using from target_sat
Solution
Direct Imports Removed: We've cleaned up our codebase by eliminating direct imports from the Nailgun package.
Using target_sat, module_target_sat, class_target_sat: Modified the API calls and using the target_sat.api. pattern for all API calls.