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

[NEW] Edit user permissions #7309

Merged
merged 11 commits into from
Jul 27, 2017
Merged

[NEW] Edit user permissions #7309

merged 11 commits into from
Jul 27, 2017

Conversation

MartinSchoeler
Copy link
Contributor

@MartinSchoeler MartinSchoeler commented Jun 22, 2017

@RocketChat/core

Closes #3656 , #3867 and #1629

captura de tela 2017-06-22 as 16 02 24

This adds the ability to change the user permissions on the user edit tab, also added a better styling to it

TODO:

  • Remove the non user scoped roles from the list

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-7309 June 22, 2017 19:09 Inactive
@@ -32,26 +35,50 @@ Template.userEdit.events({
'click .cancel'(e, t) {
e.stopPropagation();
e.preventDefault();
return t.cancel(t.find('form'));
Template.instance().roles.set([]);
Copy link
Contributor

Choose a reason for hiding this comment

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

you can use t instead of Template.instance(), right?

e.preventDefault();
let roles = Template.instance().roles.get();
roles = roles.filter(el => el !== this.valueOf());
Template.instance().roles.set(roles);
Copy link
Contributor

Choose a reason for hiding this comment

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

you can add a second parameter (t) on this event and use that here instead of Template.instance()

},

'click #randomPassword'(e) {
e.stopPropagation();
e.preventDefault();
return $('#password').val(Random.id());
$('#password').val(Random.id());
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@rodrigok rodrigok merged commit 7fa2d6e into develop Jul 27, 2017
@rodrigok rodrigok deleted the edit-user-permissions branch July 27, 2017 20:36
@rodrigok rodrigok added this to the 0.58.0 milestone Jul 27, 2017
@MartinSchoeler MartinSchoeler mentioned this pull request Aug 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change role from user edit flyout
4 participants