Skip to content

[NEW] users.setActiveStatus endpoint in rest api#13443

Merged
MarcosSpessatto merged 7 commits intoRocketChat:developfrom
thayannevls:add-users.setActiveStatus-endpoint-rest-api
Feb 16, 2019
Merged

[NEW] users.setActiveStatus endpoint in rest api#13443
MarcosSpessatto merged 7 commits intoRocketChat:developfrom
thayannevls:add-users.setActiveStatus-endpoint-rest-api

Conversation

@thayannevls
Copy link
Copy Markdown
Contributor

Closes #13425

Add new endpoint that allows to set the user Active Status using the REST API

@MarcosSpessatto
Copy link
Copy Markdown
Contributor

@thayannevls Can you please add some tests and docs for this endpoint?

.end(done);
});
});
});
Copy link
Copy Markdown
Contributor

@MarcosSpessatto MarcosSpessatto Feb 14, 2019

Choose a reason for hiding this comment

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

@thayannevls can you add a test case for the user trying update your own status?

d-gubert
d-gubert previously approved these changes Feb 14, 2019
@d-gubert d-gubert dismissed their stale review February 14, 2019 15:37

The branch has conflicts, should approve when they're resolved

@MarcosSpessatto
Copy link
Copy Markdown
Contributor

MarcosSpessatto commented Feb 15, 2019

@thayannevls the tests are broken. Can you please take a look? :)

@thayannevls
Copy link
Copy Markdown
Contributor Author

@MarcosSpessatto Fixed it. Can you check?

@MarcosSpessatto
Copy link
Copy Markdown
Contributor

Thank you @thayannevls 👍

@MarcosSpessatto MarcosSpessatto merged commit 6c206aa into RocketChat:develop Feb 16, 2019
@rodrigok rodrigok mentioned this pull request Apr 28, 2019
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.

3 participants