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

Cannot mass edit device types to 0U #1311

Closed
jakubkrysl opened this issue Jan 27, 2022 · 3 comments · Fixed by #1611
Closed

Cannot mass edit device types to 0U #1311

jakubkrysl opened this issue Jan 27, 2022 · 3 comments · Fixed by #1611
Assignees
Labels
type: bug Something isn't working as expected

Comments

@jakubkrysl
Copy link

Environment

  • Python version: 3.6.15
  • Nautobot version: 1.2.4

0U is valid height when creating or editing a device type, but mass editing does not allow this.

Steps to Reproduce

  1. Create 2 device types
  2. Mass edit them to set U height to 0U

Expected Behavior

Set to 0U

Observed Behavior

Please select a value that is no less than 1.

@bryanculver
Copy link
Member

Confirmed. Thanks for reporting this @Thetacz.

@briddo briddo self-assigned this Mar 29, 2022
@briddo
Copy link
Contributor

briddo commented Mar 30, 2022

This requires two tests which both set the u_height to 0:

  • A test which expects an error on submission of the bulk edit form because of rack position validation
  • A test which should validate a form submission of the bulk edit form

@jathanism jathanism self-assigned this Mar 31, 2022
@bryanculver bryanculver linked a pull request Apr 5, 2022 that will close this issue
6 tasks
jathanism added a commit that referenced this issue Apr 5, 2022
@bryanculver
Copy link
Member

Closed in #1611

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug Something isn't working as expected
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants