Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

Remove privileged proxy for converting ids between e-mail addresses #415

Merged
merged 1 commit into from
Jul 1, 2016

Conversation

jcscottiii
Copy link
Contributor

Originally, you needed the user id to set roles. Back when this was
corrected the only way to do that was to query for the user info first
by e-mail address to get the id. Now, there are APIs that make it
possible to set roles by e-mail address. This patch removes the need
for the privileged query call.

Example of the new by Username apis to use:
image

Originally, you needed the user id to set roles. Back when this was
corrected the only way to do that was to query for the user info first
by e-mail address to get the id. Now, there are APIs that make it
possible to set roles by e-mail address. This patch removes the need
for the privileged query call
@jcscottiii
Copy link
Contributor Author

@msecret

@jcscottiii
Copy link
Contributor Author

jcscottiii commented Jul 1, 2016

I figured that this is the time to do it since the ability to invite someone to an org via e-mail does not exist in the new deck. so when we finally do it, we can use the new api.

image

Right now, we just point people to docs.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.9%) to 70.958% when pulling 0e218bb on remove_priv_calls into da6c5c0 on staging-alpha.

@msecret msecret merged commit cba601c into staging-alpha Jul 1, 2016
@msecret msecret deleted the remove_priv_calls branch July 1, 2016 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants