Skip to content
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

update roles in create user modal if new role created #187

Conversation

aixart12
Copy link
Contributor

@aixart12 aixart12 commented Jan 25, 2024

  • "Create User Modal " : disabled the create button if password is created
  • update the role in create user modal if user add new role
  • change the create role modal title

Copy link
Contributor

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request

@aixart12
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@nitisht nitisht requested a review from balaji-jr January 25, 2024 02:59
@aixart12
Copy link
Contributor Author

#180

Copy link
Contributor

@nikhilsinhaparseable nikhilsinhaparseable left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested all three fixes, looks good

@nikhilsinhaparseable
Copy link
Contributor

@aixart12 CLA bot expects the CLA to be signed by the same user who pushed the commit, please sign the CLA with the same user @dhruv-vendo

@aixart12
Copy link
Contributor Author

closing due to pushed with non personal git-hub account

@aixart12 aixart12 closed this Jan 25, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jan 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants