Skip to content

Conversation

@liu-samuel
Copy link

@liu-samuel liu-samuel commented Oct 8, 2024

Fixes: #6907

Converts user role form from haml to React and add cypress testing

Before:
Screenshot 2025-04-30 at 11 38 58 AM
Screenshot 2025-04-30 at 11 40 22 AM

After:
Screenshot 2025-04-30 at 11 35 36 AM
Screenshot 2025-04-30 at 11 36 13 AM

@GilbertCherrie
Copy link
Member

@liu-samuel Can you please resolve the conflicts here

@liu-samuel liu-samuel force-pushed the substring-check branch 4 times, most recently from 0c02479 to b2895ab Compare November 8, 2024 21:00
@miq-bot
Copy link
Member

miq-bot commented Apr 30, 2025

Checked commits liu-samuel/manageiq-ui-classic@9928380~...3c22295 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.62.0, and yamllint
7 files checked, 5 offenses detected

app/views/ops/_rbac_role_details.html.haml

  • ⚠️ - Line 10 - Layout/ArgumentAlignment: Align the arguments of a method call if they span more than one line.
  • ⚠️ - Line 7 - Avoid using instance variables in partials views
  • ⚠️ - Line 7 - Line is too long. [91/80]
  • ⚠️ - Line 8 - Avoid using instance variables in partials views
  • ⚠️ - Line 9 - Avoid using instance variables in partials views

@miq-bot
Copy link
Member

miq-bot commented May 27, 2025

This pull request is not mergeable. Please rebase and repush.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Form conversion: Configuration Access Control/Roles/Configuration/Add a new role, Edit

4 participants