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

Added possibility to send chat messages to personal chats / group / meeting chats #816

Merged
merged 7 commits into from
Apr 25, 2022

Conversation

MathijsVerbeeck
Copy link
Contributor

Added the "IMe" interface to query the user details and list the user chats
Added possibility to create a new OneOnOne or Group chat
Added possibility to send messages to a personal OneOnOne chat, Group chat or meeting chat

Issue ref: #657

@MathijsVerbeeck
Copy link
Contributor Author

MathijsVerbeeck commented Apr 8, 2022

In case of wanting to run the tests in "ChatTests.cs", you'll have to make sure to replace the values in "ChatTests.cs" at line 19 & 20

private readonly string UserId1 = "a857e888-b602-4790-86d9-3dca2109449e"; private readonly string UserId2 = "8323f7fe-e8a4-46c4-b5ea-f4864887d160";

to userIds existing in your Azure AD Environment.

Also you'll have to make sure that a meeting chat is available for the user or the tests for the meetings in "ChatMessageTests.cs" will fail.

@jansenbe jansenbe self-assigned this Apr 9, 2022
@jansenbe jansenbe added the area: model 📐 Related to the core SDK models label Apr 9, 2022
@jansenbe
Copy link
Contributor

jansenbe commented Apr 9, 2022

Wow! Nice job again @MathijsVerbeeck , congrats for delivering this one so fast! I'll process this one in the week of the 19th as I'm OOF next week. Maybe I can even process before, but just setting expectations on the possible delay.

@jansenbe
Copy link
Contributor

@MathijsVerbeeck : took a while, but finally cleaned up my vacation backlog and can spent some time on this one :-)

@MathijsVerbeeck
Copy link
Contributor Author

MathijsVerbeeck commented Apr 25, 2022

@jansenbe No rush. I'm currently having a busy period at work/real life as well :-)

@jansenbe jansenbe merged commit 33d5933 into pnp:dev Apr 25, 2022
@MathijsVerbeeck MathijsVerbeeck deleted the sendchatmessagetouser branch May 8, 2022 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: model 📐 Related to the core SDK models
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants