Get events for multiple users in a single request #170
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
Thanks for this very useful gem.
There is a little issue regarding user availability. The
get_user_availability
method takes as parameter an array of email addresses. That's needed when one wants to get availabilities of multiple users through a single request to the Exchange server. But from my understanding, this method only returns availability for the first user in the array.This commit proposes a way to return availability for all users specified within the array. Regarding the result, since Exchange do no insert any user id within its response, availabilities are returned with a
user
value that contains the index of the user in the array passed as parameter.It might probably be more efficient to parse directly the XML response from Exchange with Nokigiri (missing time to do so).
Thanks again for the library.
Fabrice.