Skip to content

Add permissions denied SKU to DomainPermissions#2486

Merged
JamesBurnside merged 22 commits intomainfrom
jaburnsi/update-permission-denied-ui
Nov 4, 2022
Merged

Add permissions denied SKU to DomainPermissions#2486
JamesBurnside merged 22 commits intomainfrom
jaburnsi/update-permission-denied-ui

Conversation

@JamesBurnside
Copy link
Copy Markdown
Member

What

  • Add Denied type with corresponding strings and icons

Note: We have BrowserPermissions dialogs that are similar, convergence between these will happen in a future PR. This PR just adds the new SKU to the domain permissions modal.

Why

Newer call readiness designs, continuation of #2485

How Tested

Added SKU option to storybook:
image

CameraAndMicrophone Camera Microphone
image image image

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 3, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 3, 2022

Calling bundle size is increased❗.

  • Current size: 5336496
  • Base size: 5333732
  • Diff size: 2764

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 3, 2022

Chat bundle size is increased❗.

  • Current size: 5445389
  • Base size: 5442838
  • Diff size: 2551

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 3, 2022

CallWithChat bundle size is increased❗.

  • Current size: 5724079
  • Base size: 5721315
  • Diff size: 2764

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 3, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 3, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 4, 2022

Comment thread packages/react-components/src/localization/locales/en-US/strings.json Outdated
Comment thread packages/react-components/src/theming/icons.tsx Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 4, 2022

@JamesBurnside JamesBurnside force-pushed the jaburnsi/domainPermissionsUIUpdates branch from a33e804 to 5c4f9b9 Compare November 4, 2022 17:42
@JamesBurnside JamesBurnside force-pushed the jaburnsi/domainPermissionsUIUpdates branch from 5c4f9b9 to 18f6929 Compare November 4, 2022 17:47
…tes' into jaburnsi/update-permission-denied-ui
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 4, 2022

Base automatically changed from jaburnsi/domainPermissionsUIUpdates to main November 4, 2022 18:14
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 4, 2022

@JamesBurnside JamesBurnside enabled auto-merge (squash) November 4, 2022 18:29
@JamesBurnside JamesBurnside merged commit 363401b into main Nov 4, 2022
@JamesBurnside JamesBurnside deleted the jaburnsi/update-permission-denied-ui branch November 4, 2022 18:35
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