Skip to content

Commit

Permalink
Merge branch 'main' into jeklouda/release_2.31.0_proxy
Browse files Browse the repository at this point in the history
  • Loading branch information
jekloudaMSFT committed Nov 15, 2024
2 parents 75821b0 + 600562f commit 9d866cb
Show file tree
Hide file tree
Showing 10 changed files with 26 additions and 40 deletions.
10 changes: 5 additions & 5 deletions apps/teams-test-app/e2e-test-data/authentication.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,19 +19,19 @@
"expectedTestAppValue": "Success: {\"oid\":\"mockoid\",\"tid\":\"mocktid\",\"upn\":\"mockupn\",\"loginHint\":\"mockLoginHint\",\"displayName\":\"mockName\",\"dataResidency\":\"public\"}"
},
{
"title": "getUser API Call - Failure (Not full trust)",
"title": "getUser API Call - Failure (Not microsoft owned)",
"version": ">2.25.0",
"hostSdkVersion": {
"web": "<=3.0.2"
},
"type": "callResponse",
"platformsExcluded": ["iOS"],
"boxSelector": "#box_getUser",
"testUrlParams": [["appDefOverrides", "{\"isFullTrustApp\": false}"]],
"testUrlParams": [["appDefOverrides", "{\"isFullTrustApp\": false, \"isMicrosoftOwned\": false}"]],
"expectedTestAppValue": "Error: Error: Error returned, code = 500, message = App does not have the required permissions for this operation"
},
{
"title": "getUser API Call - Failure (Not full trust and not Microsoft owned)",
"title": "getUser API Call - Failure (Not Microsoft owned)",
"version": ">2.25.0",
"hostSdkVersion": {
"web": ">3.0.2"
Expand All @@ -43,7 +43,7 @@
"expectedTestAppValue": "Error: Error: Error returned, code = 500, message = App does not have the required permissions for this operation"
},
{
"title": "getUser API Call - Success (Full trust app)",
"title": "getUser API Call - Success (Full trust app - backwards compatibility)",
"version": ">2.25.0",
"hostSdkVersion": {
"web": ">3.0.2"
Expand All @@ -55,7 +55,7 @@
"expectedTestAppValue": "{\"oid\":\"mockoid\",\"tid\":\"mocktid\",\"upn\":\"mockupn\",\"loginHint\":\"mockLoginHint\",\"displayName\":\"mockName\",\"dataResidency\":\"public\"}"
},
{
"title": "getUser API Call - Success (Full trust and Microsoft owned app)",
"title": "getUser API Call - Success (Microsoft owned app)",
"version": ">2.25.0",
"hostSdkVersion": {
"web": ">3.0.2"
Expand Down
4 changes: 2 additions & 2 deletions apps/teams-test-app/e2e-test-data/files.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,13 @@
"expectedTestAppValue": "[{\"id\":\"testId\",\"title\":\"testTitle\",\"folderId\":\"testFolderId\",\"providerType\":2,\"providerCode\":\"GOOGLEDRIVE\",\"ownerDisplayName\":\"testOwnerDisplayName\"}]"
},
{
"title": "getCloudStorageFolders API Call - Failure (Not Full Trust)",
"title": "getCloudStorageFolders API Call - Failure (Not microsoft owned)",
"type": "callResponse",
"platformsExcluded": ["iOS"],
"version": ">2.0.0-beta.5",
"boxSelector": "#box_getCloudStorageFolders2",
"inputValue": "testChannelId",
"testUrlParams": [["appDefOverrides", "{\"isFullTrustApp\": false}"]],
"testUrlParams": [["appDefOverrides", "{\"isFullTrustApp\": false, \"isMicrosoftOwned\": false}"]],
"expectedTestAppValue": "{\"errorCode\":500,\"message\":\"App does not have the required permissions for this operation\"}"
},
{
Expand Down
4 changes: 2 additions & 2 deletions apps/typed-dependency-tester/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@
"description": "A tester to check if types in the dependency array will cause errors",
"version": "0.0.1",
"scripts": {
"build": "pnpm i && pnpm copy && pnpm tsc && pnpm clean",
"copy": "cp ./node_modules/@microsoft/teams-js/dist/umd/MicrosoftTeams.d.ts ./ || xcopy .\\node_modules\\@microsoft\\teams-js\\dist\\umd\\MicrosoftTeams.d.ts .\\ /Y",
"build": "pnpm i && pnpm copy && pnpm clean",
"copy": "cp -r ./node_modules/@microsoft/teams-js/dist/esm/packages/teams-js/dts ./ || xcopy .\\node_modules\\@microsoft\\teams-js\\dist\\esm\\packages\\teams-js\\dts .\\ /s /Y",
"clean": "rimraf node_modules && rimraf MicrosoftTeams.d.ts"
},
"dependencies": {},
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "Fixed bugs regarding `EduType`not being exported, and enum typings",
"packageName": "@microsoft/teams-js",
"email": "[email protected]",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "Fixed a bug causing `buffer` polyfill to stil be included",
"packageName": "@microsoft/teams-js",
"email": "[email protected]",
"dependentChangeType": "patch"
}
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,6 @@
"prettier": "^3.3.2",
"rimraf": "^5.0.7",
"rollup": "^4.24.4",
"rollup-plugin-dts": "^6.1.1",
"rollup-plugin-polyfill-node": "^0.13.0",
"shx": "^0.3.4",
"style-loader": "^3.3.4",
Expand Down
2 changes: 1 addition & 1 deletion packages/teams-js/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
"url": "https://github.com/OfficeDev/microsoft-teams-library-js"
},
"main": "./dist/umd/MicrosoftTeams.min.js",
"typings": "./dist/umd/MicrosoftTeams.d.ts",
"typings": "./dist/esm/packages/teams-js/dts/index.d.ts",
"module": "./dist/esm/packages/teams-js/src/index.js",
"scripts": {
"build": "pnpm clean && pnpm lint && pnpm build-rollup && pnpm build-webpack && pnpm docs:validate",
Expand Down
12 changes: 1 addition & 11 deletions packages/teams-js/rollup.config.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import replace from '@rollup/plugin-replace';
import terser from '@rollup/plugin-terser';
import typescript from '@rollup/plugin-typescript';
import { readFileSync } from 'fs';
import dts from 'rollup-plugin-dts';
import nodePolyfills from 'rollup-plugin-polyfill-node';

const packageJson = JSON.parse(readFileSync('./package.json', 'utf-8'));
Expand Down Expand Up @@ -43,7 +42,7 @@ export default [
typescript(),
json(),
commonjs(),
nodePolyfills(),
nodePolyfills({ exclude: ['node_modules/**'] }),
],
treeshake: {
moduleSideEffects: [
Expand All @@ -59,13 +58,4 @@ export default [
],
},
},
{
input: './dist/esm/packages/teams-js/dts/index.d.ts',
output: {
file: 'dist/umd/MicrosoftTeams.d.ts',
format: 'es',
sourcemap: false,
},
plugins: [dts()],
},
];
1 change: 1 addition & 0 deletions packages/teams-js/src/public/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ export {
DeepLinkParameters,
DialogInfo,
DialogSize,
EduType,
ErrorCode,
FileOpenPreference,
FrameContext,
Expand Down
18 changes: 0 additions & 18 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 9d866cb

Please sign in to comment.