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 #26052 - add missing IDs to Layout #6551
Conversation
Issues: #26052 |
I'm using the |
Dashboard url is
maybe we should hardcode the attributes in the server @ohadlevy ? |
@lhellebr you may want to have a look at this as I'm not sure what the IDs were prior to the react navigation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of comments -
- This does not restore the same ids for the menu items as before. The original id was generated by the item name - https://github.com/theforeman/foreman/blob/1.19-stable/app/helpers/home_helper.rb#L41. This means that changing it to be based on the url will still cause multiple failures for QE automation.
- The data
aid
attribute should no longer be used by automation - see Fixes #23168 - undefined method `split' for nil:NilClass #5712 (comment) and SatelliteQE/airgun@8438e72 which dropped it completely, so we don't need to continue using this attribute which was only used for QE automation.
f978bbb
to
ef711e9
Compare
removed |
ef711e9
to
5331b9c
Compare
5331b9c
to
051b9d6
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See inline comment; this also needs updating the react-patternfly rpm version
@tbrisker can we merge? |
Thanks @glekner! I didn't notice that you pushed the update a couple of weeks ago. |
No description provided.