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

Error when saving dynamic groups is not displayed intuitively. #3212

Closed
Tracked by #1565
ebusto opened this issue Feb 2, 2023 · 3 comments · Fixed by #4359
Closed
Tracked by #1565

Error when saving dynamic groups is not displayed intuitively. #3212

ebusto opened this issue Feb 2, 2023 · 3 comments · Fixed by #4359
Labels
type: bug Something isn't working as expected

Comments

@ebusto
Copy link

ebusto commented Feb 2, 2023

Environment

  • Nautobot version: 1.5.9
  • Python version: 3.10
  • Database platform, version: PostgreSQL 14
  • Middleware(s): N/A

Steps to Reproduce

  1. Create a parent dynamic group with a child and a filter.

Expected Behavior

As this isn't allowed, the error should be displayed after attempting to save the new dynamic group: "A parent group may have either a filter or child groups, but not both. Clear the parent filter and try again."

Observed Behavior

The error is lost, and instead "A parent group may have either a filter or child groups, but not both. Clear the parent filter and try again." is the only visible error message.

@jathanism
Copy link
Contributor

Hmm... Are you trying to do this in the UI? And I'm confused a bit, as you seem to be indicating that the expected and observed behavior are the same? Did you copy/paste the wrong error in one of the cases? Thanks in advance!

@jathanism
Copy link
Contributor

I went to reproduce and created an initial group called "child1". In the UI when I go to create "parent1", set a filter has_primary_ip=False and assign "child1" to it, I get the errors, but the expected error is hidden under the "Child Groups" nav tab under "Filter Options". See:

image

Is this the confusion? Because I'll admit, this UX is kind of not great.

@ebusto
Copy link
Author

ebusto commented Feb 2, 2023

@jathanism You're right, I flipped the expected and observed. Clearly I needed more coffee.

I didn't think to check the "child groups" tab for the error. I simply looked below as instructed, and saw a form without any error indications.

If the UI is improved a smidge so this error is front and center, that should reduce confusion.

@bryanculver bryanculver added the type: bug Something isn't working as expected label Feb 3, 2023
@glennmatthews glennmatthews changed the title Error when saving dynamic groups is lost. Error when saving dynamic groups is not displayed intuitively. Feb 3, 2023
@bryanculver bryanculver mentioned this issue Feb 6, 2023
15 tasks
@DistantVoyager DistantVoyager linked a pull request Aug 30, 2023 that will close this issue
3 tasks
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 30, 2023
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
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants