-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Add system jobs interface w/ configurable data retention #9224
Add system jobs interface w/ configurable data retention #9224
Conversation
awx/ui_next/src/screens/ManagementJob/ManagementJobList/ManagementJobList.jsx
Show resolved
Hide resolved
Build succeeded.
|
awx/ui_next/src/screens/ManagementJob/ManagementJobList/LaunchManagementPrompt.jsx
Show resolved
Hide resolved
@jakemcdermott we had some difficulties getting an environment set up. Our only point of feedback so far is that it might make sense to get rid of advanced search from this view |
@jakemcdermott I took a look and it looks good. It sounds like you want to make a few more changes (table conversion and removing the launch prompt). Let me know when the dust has settled and I'll take another look. |
37e6660
to
c81c9fe
Compare
Build succeeded.
|
import NotificationsMixin from '../mixins/Notifications.mixin'; | ||
import SchedulesMixin from '../mixins/Schedules.mixin'; | ||
|
||
const Mixins = SchedulesMixin(NotificationsMixin(Base)); |
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.
🆒
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.
Build succeeded.
|
Build failed.
|
1839cc4
to
3af02ae
Compare
* Migate management jobs list to tables * Use cancel link variant for consistency with other prompts * Add basic test coverage for sysjobs * Remove select-all from mgmt jobs * Remove unneeded component variables * Fix missing schedule breadcrumb * Optimize data fetching with useCallback
Co-authored-by: Tiago Góes <tiago.goes2009@gmail.com>
3af02ae
to
218b978
Compare
@one-t I think this should be fixed now - Can you try it w/ the latest? |
Build succeeded.
|
Build succeeded (gate pipeline).
|
Mockups: https://tower-mockups.testing.ansible.com/patternfly/mgmt-jobs/mgmt-jobs
Related: #8488
SUMMARY