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

fixes #8087 - Locations - update to allow proper use of nested locations #4763

Merged
merged 1 commit into from Oct 27, 2014

Conversation

bbuckingham
Copy link
Member

Without this commit, the user can click 'Nest' to created a nested
location; however, they cannot actually assign a parent location.
As a result, nesting really isn't functional.

This commit will conditionally remove the 'parent' for organization
taxonomies, but leave them in place on the location taxonomies.

This can be tested using the following:

  • new & edit org - confirm there is no 'parent'
  • new, edit, nest location - confirm that user can select a 'parent'

… locations

Without this commit, the user can click 'Nest' to created a nested
location; however, they cannot actually assign a parent location.
As a result, nesting really isn't functional.

This commit will conditionally remove the 'parent' for organization
taxonomies, but leave them in place on the location taxonomies.

This can be tested using the following:
- new & edit org - confirm there is no 'parent'
- new, edit, nest location - confirm that user can select a 'parent'
@stbenjam
Copy link
Contributor

ACK

bbuckingham added a commit that referenced this pull request Oct 27, 2014
fixes #8087 - Locations - update to allow proper use of nested locations
@bbuckingham bbuckingham merged commit 38ac851 into Katello:master Oct 27, 2014
@bbuckingham bbuckingham deleted the issue-8087 branch October 27, 2014 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants