-
Notifications
You must be signed in to change notification settings - Fork 283
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
feat: add quick messaging to fluent person-card #1958
feat: add quick messaging to fluent person-card #1958
Conversation
Thank you for creating a Pull Request @Mnickii. This is a checklist for the PR reviewer(s) to complete before approving and merging this PR:
|
The updated storybook is available here |
|
packages/mgt-components/src/components/mgt-person-card/mgt-person-card.scss
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-person-card/mgt-person-card.graph.ts
Outdated
Show resolved
Hide resolved
The updated storybook is available here |
From an experience standpoint (I can't test if this is really "sent" because of the sandbox):
Example from Teams |
packages/mgt-components/src/components/mgt-person-card/mgt-person-card.ts
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-person-card/mgt-person-card.ts
Outdated
Show resolved
Hide resolved
The updated storybook is available here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Messages don't work on any of the story I've been trying. Rest looks good to me!
The updated storybook is available here |
packages/mgt-components/src/components/mgt-contact/mgt-contact.scss
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-contact/mgt-contact.ts
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-organization/mgt-organization.scss
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-profile/mgt-profile.scss
Outdated
Show resolved
Hide resolved
@sebastienlevert they're sending for me, but with no affordance in the UI that they were sent. |
@gavinbarron I didn't mean sending messages. I mean I can't see the tab with the emails between me and the user. Also, good catch, I get the same. I noted the issue in an earlier comment #1958 (comment) |
Ahh, gotcha, so many types of messages. |
I've taken a look at this and I think that it's fine. This section renders messages that the user viewing the card has received from the user being rendered. It uses Mail.ReadBasic.
The default user in the MockProvider only has 1 email that isn't Analytics/Viva/Outlook sent, and it's from themselves! In my test tenant, signed in with the admin account I see emails from Megan that she sent to a Group the admin is part of: The Files Tab has a similar behavior where it doesn't show if there are no files to show. |
Change chat input background color Disable send icon while sending
…thub.com/microsoftgraph/microsoft-graph-toolkit into mnickii/person-card-fluent-quick-message
The updated storybook is available here |
packages/mgt-components/src/components/mgt-person-card/mgt-person-card.ts
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-person-card/mgt-person-card.ts
Outdated
Show resolved
Hide resolved
packages/mgt-components/src/components/mgt-person-card/mgt-person-card.ts
Outdated
Show resolved
Hide resolved
…son-card.ts Co-authored-by: Gavin Barron <gavinbarron@microsoft.com>
…son-card.ts Co-authored-by: Gavin Barron <gavinbarron@microsoft.com>
@Mnickii where are we with this? It seems to be pending latest changes to be able to merge? |
@sebastienlevert It should be ready to go now |
The updated storybook is available here |
The updated storybook is available here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ship it!
The updated storybook is available here |
1 similar comment
The updated storybook is available here |
Closes #1957
PR Type
Feature
Description of the changes
Adds a quick message section to the person-card overview.
PR checklist
yarn build
) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)yarn setLicense
)Other information