Skip to content

Convert role form from ruby into React #9281

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

liu-samuel
Copy link
Contributor

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

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.

4 participants