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

Add job settings form and unit tests #8661

Merged

Conversation

marshmalien
Copy link
Member

SUMMARY

Issue: #8290
This PR adds the job category settings form.

Screen Shot 2020-11-23 at 3 28 32 PM

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
  • UI

@marshmalien marshmalien added state:needs_review type:feature prioritized on a feature board labels Nov 23, 2020
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

Copy link
Contributor

@nixocio nixocio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@unlikelyzero
Copy link

Will only require manual testing and re-enabling of TBI stubs

@unlikelyzero
Copy link

Not a requirement, but this would be a great time to tackle this https://github.com/ansible/tower-qa/issues/5813 to get some global state tests online

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@unlikelyzero
Copy link

@marshmalien it looks like the order of fields is different between edit page and detail page

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

Copy link
Contributor

@tiagodread tiagodread left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, it worked as expected updated some test here https://github.com/ansible/tower-qa/pull/5918

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 89646e7 into ansible:devel Jan 29, 2021
@tiagodread tiagodread mentioned this pull request Jan 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:ui type:feature prioritized on a feature board
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants