Skip to content
This repository has been archived by the owner on Oct 17, 2024. It is now read-only.

Commit

Permalink
Bug fix for availability count mismatch
Browse files Browse the repository at this point in the history
People presence(status) is coming in the exactly same case we have stored in the onlinePresenceOptions list i.e Busy, DoNotDistrub, Available but when we are making it upper case, the contains function was never returning true as it is case sensitive. So, the count was not increasing.
  • Loading branch information
HunaidHanfee-MSFT authored Oct 5, 2020
1 parent 507d391 commit 6532265
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ public async Task<int> GetDistributionListMembersOnlineCountAsync(string groupId
for (int i = 0; i < peoplePresenceResults.Count; i++)
{
this.memoryCache.Set(peoplePresenceResults[i].Id, peoplePresenceResults[i], options);
if (this.onlinePresenceOptions.Contains(peoplePresenceResults[i].Availability.ToUpper()))
if (this.onlinePresenceOptions.Contains(peoplePresenceResults[i].Availability))
{
onlineMembersCount++;
}
Expand Down

0 comments on commit 6532265

Please sign in to comment.