Skip to content

Commit fe7eeec

Browse files
[Call Readiness] Environmentstrings api fix (#2635)
* update api to be optional * build API * Change files * Duplicate change files for beta release * build stable
1 parent 0b5056c commit fe7eeec

9 files changed

Lines changed: 24 additions & 29 deletions

File tree

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
{
2+
"type": "prerelease",
3+
"comment": "Update the different environment info modals so the strings are optional props. ",
4+
"packageName": "@azure/communication-react",
5+
"email": "94866715+dmceachernmsft@users.noreply.github.com",
6+
"dependentChangeType": "patch"
7+
}
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
{
2+
"type": "prerelease",
3+
"comment": "Update the different environment info modals so the strings are optional props. ",
4+
"packageName": "@azure/communication-react",
5+
"email": "94866715+dmceachernmsft@users.noreply.github.com",
6+
"dependentChangeType": "patch"
7+
}

packages/communication-react/review/beta/communication-react.api.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3069,7 +3069,7 @@ export const UnsupportedBrowser: (props: UnsupportedBrowserProps) => JSX.Element
30693069
// @beta
30703070
export interface UnsupportedBrowserProps {
30713071
onTroubleshootingClick?: () => void;
3072-
strings: UnsupportedBrowserStrings;
3072+
strings?: UnsupportedBrowserStrings;
30733073
}
30743074

30753075
// @beta
@@ -3103,7 +3103,7 @@ export const UnsupportedOperatingSystem: (props: UnsupportedOperatingSystemProps
31033103
// @beta
31043104
export interface UnsupportedOperatingSystemProps {
31053105
onTroubleshootingClick?: () => void;
3106-
strings: UnsupportedOperatingSystemStrings;
3106+
strings?: UnsupportedOperatingSystemStrings;
31073107
}
31083108

31093109
// @beta

packages/react-components/review/beta/react-components.api.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1582,7 +1582,7 @@ export const UnsupportedBrowser: (props: UnsupportedBrowserProps) => JSX.Element
15821582
// @beta
15831583
export interface UnsupportedBrowserProps {
15841584
onTroubleshootingClick?: () => void;
1585-
strings: UnsupportedBrowserStrings;
1585+
strings?: UnsupportedBrowserStrings;
15861586
}
15871587

15881588
// @beta
@@ -1616,7 +1616,7 @@ export const UnsupportedOperatingSystem: (props: UnsupportedOperatingSystemProps
16161616
// @beta
16171617
export interface UnsupportedOperatingSystemProps {
16181618
onTroubleshootingClick?: () => void;
1619-
strings: UnsupportedOperatingSystemStrings;
1619+
strings?: UnsupportedOperatingSystemStrings;
16201620
}
16211621

16221622
// @beta

packages/react-components/review/stable/react-components.api.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1337,7 +1337,7 @@ export const UnsupportedBrowser: (props: UnsupportedBrowserProps) => JSX.Element
13371337
// @beta
13381338
export interface UnsupportedBrowserProps {
13391339
onTroubleshootingClick?: () => void;
1340-
strings: UnsupportedBrowserStrings;
1340+
strings?: UnsupportedBrowserStrings;
13411341
}
13421342

13431343
// @beta
@@ -1371,7 +1371,7 @@ export const UnsupportedOperatingSystem: (props: UnsupportedOperatingSystemProps
13711371
// @beta
13721372
export interface UnsupportedOperatingSystemProps {
13731373
onTroubleshootingClick?: () => void;
1374-
strings: UnsupportedOperatingSystemStrings;
1374+
strings?: UnsupportedOperatingSystemStrings;
13751375
}
13761376

13771377
// @beta

packages/react-components/src/components/UnsupportedBrowser.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export interface UnsupportedBrowserProps {
2929
/** Handler to perform an action when the help link is actioned */
3030
onTroubleshootingClick?: () => void;
3131
/** String overrides for the component */
32-
strings: UnsupportedBrowserStrings;
32+
strings?: UnsupportedBrowserStrings;
3333
}
3434

3535
/**

packages/react-components/src/components/UnsupportedOperatingSystem.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export interface UnsupportedOperatingSystemProps {
2929
/** Handler to perform a action when the help link is actioned */
3030
onTroubleshootingClick?: () => void;
3131
/** String overrides for the component */
32-
strings: UnsupportedOperatingSystemStrings;
32+
strings?: UnsupportedOperatingSystemStrings;
3333
}
3434

3535
/**

packages/storybook/stories/INTERNAL/UnsupportedBrowser/snippets/UnsupportedEnvironmentDrawer.snippet.tsx

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,9 @@ import {
99
UnsupportedOperatingSystem
1010
} from '@internal/react-components';
1111
import React, { useState } from 'react';
12-
import { useLocale } from '../../../../../react-components/src/localization';
1312
import { MobilePreviewContainer } from '../../../MobileContainer';
1413

1514
export const UnsupportedEnvironmentDrawers: () => JSX.Element = () => {
16-
const unsupportedBrowserStrings = useLocale().strings.UnsupportedBrowser;
17-
const unsupportedBrowserVersionStrings = useLocale().strings.UnsupportedBrowserVersion;
18-
const unsupportedOperatingSystemStrings = useLocale().strings.UnsupportedOperatingSystem;
1915
const [unsupportedBrowserShowing, setUnsupportedBrowserShowing] = useState(false);
2016
const [unsupportedBrowserVersionShowing, setUnsupportedBrowserVersionShowing] = useState(false);
2117
const [unsupportedOperatingSystemShowing, setUnsupportedOperatingSystemShowing] = useState(false);
@@ -38,10 +34,7 @@ export const UnsupportedEnvironmentDrawers: () => JSX.Element = () => {
3834
)}
3935
{unsupportedBrowserShowing && (
4036
<_DrawerSurface onLightDismiss={onLightDismissTriggeredUnsupportedBrowser}>
41-
<UnsupportedBrowser
42-
onTroubleshootingClick={() => alert('clicked trouble shooting link')}
43-
strings={unsupportedBrowserStrings}
44-
/>
37+
<UnsupportedBrowser onTroubleshootingClick={() => alert('clicked trouble shooting link')} />
4538
</_DrawerSurface>
4639
)}
4740
</MobilePreviewContainer>
@@ -61,7 +54,6 @@ export const UnsupportedEnvironmentDrawers: () => JSX.Element = () => {
6154
<_DrawerSurface onLightDismiss={onLightDismissTriggeredUnsupportedBrowserVersion}>
6255
<UnsupportedBrowserVersion
6356
onTroubleshootingClick={() => alert('clicked compatibility link')}
64-
strings={unsupportedBrowserVersionStrings}
6557
onContinueAnywayClick={() => alert('you are brave arent you?')}
6658
/>
6759
</_DrawerSurface>
@@ -81,10 +73,7 @@ export const UnsupportedEnvironmentDrawers: () => JSX.Element = () => {
8173
)}
8274
{unsupportedOperatingSystemShowing && (
8375
<_DrawerSurface onLightDismiss={onLightDismissTriggeredUnsupportedOperatingSystem}>
84-
<UnsupportedOperatingSystem
85-
onTroubleshootingClick={() => alert('clicked compatibility link')}
86-
strings={unsupportedOperatingSystemStrings}
87-
/>
76+
<UnsupportedOperatingSystem onTroubleshootingClick={() => alert('clicked compatibility link')} />
8877
</_DrawerSurface>
8978
)}
9079
</MobilePreviewContainer>

packages/storybook/stories/INTERNAL/UnsupportedBrowser/snippets/UnsupportedEnvironmentModal.snippet.tsx

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,7 @@ import { Modal, PrimaryButton, Stack } from '@fluentui/react';
55
import { UnsupportedBrowser, UnsupportedBrowserVersion, UnsupportedOperatingSystem } from '@internal/react-components';
66
import React, { useState } from 'react';
77

8-
import { useLocale } from '../../../../../react-components/src/localization';
9-
108
export const UnsupportedEnvironmentModals: () => JSX.Element = () => {
11-
const unsupportedBrowserStrings = useLocale().strings.UnsupportedBrowser;
12-
const unsupportedBrowserVersionStrings = useLocale().strings.UnsupportedBrowserVersion;
13-
const unsupportedBrowserOperatingSystem = useLocale().strings.UnsupportedOperatingSystem;
149
const [unsupportedBrowserModalOpen, setUnsupportedBrowserModalOpen] = useState<boolean>(false);
1510
const [unsupportedBrowserVersionModalOpen, setUnsupportedBrowserVersionModalOpen] = useState<boolean>(false);
1611
const [unsupportedOperatingSystemModalOpen, setUnsupportedOperatingSystemModalOpen] = useState<boolean>(false);
@@ -42,15 +37,13 @@ export const UnsupportedEnvironmentModals: () => JSX.Element = () => {
4237
</PrimaryButton>
4338
<Modal isOpen={unsupportedBrowserModalOpen} onDismiss={() => setUnsupportedBrowserModalOpen(false)}>
4439
<UnsupportedBrowser
45-
strings={unsupportedBrowserStrings}
4640
onTroubleshootingClick={() => {
4741
alert('clicked help link');
4842
}}
4943
/>
5044
</Modal>
5145
<Modal isOpen={unsupportedBrowserVersionModalOpen} onDismiss={() => setUnsupportedBrowserVersionModalOpen(false)}>
5246
<UnsupportedBrowserVersion
53-
strings={unsupportedBrowserVersionStrings}
5447
onTroubleshootingClick={() => {
5548
alert('clicked help link');
5649
}}
@@ -62,7 +55,6 @@ export const UnsupportedEnvironmentModals: () => JSX.Element = () => {
6255
onDismiss={() => setUnsupportedOperatingSystemModalOpen(false)}
6356
>
6457
<UnsupportedOperatingSystem
65-
strings={unsupportedBrowserOperatingSystem}
6658
onTroubleshootingClick={() => {
6759
alert('clicked help link');
6860
}}

0 commit comments

Comments
 (0)