Skip to content

Add parameter to hint displayName from SDK#2697

Merged
PorterNan merged 6 commits intomainfrom
jinan/add-displayName-from-sdk
Feb 21, 2023
Merged

Add parameter to hint displayName from SDK#2697
PorterNan merged 6 commits intomainfrom
jinan/add-displayName-from-sdk

Conversation

@PorterNan
Copy link
Copy Markdown
Contributor

What

Add parameter to hint displayName from SDK, so Contoso doesn't have to override it if there is any from sdk

Why

How Tested

Process & policy checklist

  • I have updated the project documentation to reflect my changes if necessary.
  • I have read the CONTRIBUTING documentation.

Is this a breaking change?

  • This change causes current functionality to break.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 1, 2023

Calling bundle size is increased❗.

  • Current size: 5576197
  • Base size: 5576166
  • Diff size: 31

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 1, 2023

Chat bundle size is not changed.

  • Current size: 5656107
  • Base size: 5656107
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 1, 2023

CallWithChat bundle size is increased❗.

  • Current size: 5962844
  • Base size: 5962813
  • Diff size: 31

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 1, 2023

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 2, 2023

Copy link
Copy Markdown
Member

@JamesBurnside JamesBurnside left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This a great solution to make Contoso know what to provide! Love it!

Comment thread packages/communication-react/review/beta/communication-react.api.md Outdated
@PorterNan PorterNan enabled auto-merge (squash) February 21, 2023 22:47
@github-actions
Copy link
Copy Markdown
Contributor

@PorterNan PorterNan merged commit 766f54d into main Feb 21, 2023
@PorterNan PorterNan deleted the jinan/add-displayName-from-sdk branch February 21, 2023 23:14
PorterNan added a commit that referenced this pull request Feb 21, 2023
* Add parameter to hint displayName from SDK
---------

Co-authored-by: Nan Jiang <jinan@microsoft.com>
edwardlee-msft pushed a commit that referenced this pull request Feb 23, 2023
* Update storybook for cte display name (#2766)

* Add code snippet for displayName override

---------
Co-authored-by: Nan Jiang <jinan@microsoft.com>
Co-authored-by: edwardlee-msft <edwardlee@microsoft.com>

* Add parameter to hint displayName from SDK (#2697)

* Add parameter to hint displayName from SDK
---------

Co-authored-by: Nan Jiang <jinan@microsoft.com>

---------

Co-authored-by: Nan Jiang <jinan@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants