Skip to content

Only add 'Remove participant' menu item present if role allows#2328

Merged
mgamis-msft merged 10 commits intomainfrom
mgamis/remove-menu-item-if-role-allows
Sep 14, 2022
Merged

Only add 'Remove participant' menu item present if role allows#2328
mgamis-msft merged 10 commits intomainfrom
mgamis/remove-menu-item-if-role-allows

Conversation

@mgamis-msft
Copy link
Copy Markdown
Contributor

What

Only add Remove participant menu item present if role allows

Why

https://skype.visualstudio.com/SPOOL/_workitems/edit/2972692
https://skype.visualstudio.com/SPOOL/_workitems/edit/2972704

How Tested

Local calling sample

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
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 12, 2022

Calling bundle size is decreased✅.

  • Current size: 5067666
  • Base size: 5067711
  • Diff size: -45

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 12, 2022

Chat bundle size is decreased✅.

  • Current size: 5193312
  • Base size: 5193321
  • Diff size: -9

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 12, 2022

CallWithChat bundle size is decreased✅.

  • Current size: 5453055
  • Base size: 5453100
  • Diff size: -45

@github-actions
Copy link
Copy Markdown
Contributor

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "ui change" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@mgamis-msft mgamis-msft merged commit 0bb131c into main Sep 14, 2022
@mgamis-msft mgamis-msft deleted the mgamis/remove-menu-item-if-role-allows branch September 14, 2022 04:46
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.

4 participants