Project

General

Profile

Bug #31791

Saving bookmark with an existing/duplicate bookmark name doesn't show a clear error

Added by Amir Fefer 5 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Search
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1920566

Description of problem:

Saving a bookmark with an existing/duplicate bookmark name doesn't show a clear error.

Version-Release number of selected component (if applicable):

Red Hat Satellite 6.9 Beta

How reproducible:

Always

Steps to Reproduce:
1. Navigate to Monitor >> Tasks in Satellite GUI
2. Enter any search query and try to save the search query as a bookmark.
3. In the "Name" field provide an existing/duplicate bookmark name.
4. Click on "Submit"

Actual results:

The "Create Bookmark" form gets closed and the following error is displayed in the top right corner of the webpage:

"Oh no! Something went wrong while submitting the form, the server returned the following error: Error: Request failed with status code 422"

This is unclear.

Expected results:

If we follow the same Reproducer steps in Red Hat Satellite 6.8, the "Create Bookmark" form is not closed and an appropriate error "has already been taken" is displayed next to the "Name" field on the form.

Additional info:


Related issues

Related to Foreman - Bug #31798: react forms create toast notifications with unclear error descriptionClosed

Associated revisions

Revision ace2b352 (diff)
Added by Amir Fefer 5 months ago

Fixes #31791 - Add duplicate name validation to bookmarks (#8304)

History

#1 Updated by The Foreman Bot 5 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8304 added

#2 Updated by Amir Fefer 5 months ago

  • Description updated (diff)

#3 Updated by Amir Fefer 5 months ago

  • Related to Bug #31798: react forms create toast notifications with unclear error description added

#4 Updated by The Foreman Bot 5 months ago

  • Fixed in Releases 2.5.0 added

#5 Updated by Amir Fefer 5 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/foreman/pull/8506 added

Also available in: Atom PDF