[AAP-80860] Fix EDA role wizard resource type dropdown and search#3322
[AAP-80860] Fix EDA role wizard resource type dropdown and search#3322amasolov wants to merge 1 commit into
Conversation
The EdaSelectResourceTypeStep parsed the OPTIONS response content_type
choices as objects ({value, display_name}) but the Django REST Framework
backend returns them as tuples ([value, display_name]). This caused
the .filter() to get undefined for every item, producing an empty
"Resource type" dropdown and blocking all EDA role assignments.
Align the EDA implementation with the AWX equivalent by using a tuple
type and array destructuring.
Additionally, switch the resource selection toolbar filter from 'name'
to 'name__icontains' for case-insensitive search (missed in ansible#3122).
Signed-off-by: Alexey Masolov <[email protected]>
Co-authored-by: Cursor <[email protected]>
|
|
After further investigation using Playwright against a live AAP 2.7 instance, the "Resource type" dropdown in the unified "Assign roles" wizard ( The unified wizard uses This suggests the original report may have been against a standalone EDA development server or older codebase where that component was still active. Leaving this PR open for team discussion, as the fix may still be valid for standalone EDA UI contexts. |
|
This PR is blocked by: ansible/eda-server-operator#331
That is correct and why I included the site header in my report (#3122 (comment)). The version of the EDA server being used is ansible/eda-server@b4030df, due to a bug somewhere after this commit because login does not work. see bug report ansible/eda-server#1504. Currently no newer version of EDA Server can be used until this bug is addressed. There is a PR open, ansible/eda-server-operator#331. However no one appears to be interested in addressing. As such no newer version of EDA server can be used to confirm if this PR is relevant/"still an issue" for EDA server, as the eda-server cant be logged on to. |



Summary
name__icontainsfor case-insensitive search (missed in [AAP-42092] Use case-insensitive filters for all toolbar text searches #3122)Type of Change
Risk Analysis - REQUIRED
Testing
Manual Testing
Relates to #3122