diff --git a/packages/drivers/odsp-driver/api-report/odsp-driver.legacy.alpha.api.md b/packages/drivers/odsp-driver/api-report/odsp-driver.legacy.alpha.api.md index e1367bf0ac9b..b56fa698af41 100644 --- a/packages/drivers/odsp-driver/api-report/odsp-driver.legacy.alpha.api.md +++ b/packages/drivers/odsp-driver/api-report/odsp-driver.legacy.alpha.api.md @@ -11,7 +11,7 @@ export function checkUrl(documentUrl: URL): DriverPreCheckInfo | undefined; export function createLocalOdspDocumentServiceFactory(localSnapshot: Uint8Array | string): IDocumentServiceFactory; // @alpha -export function createOdspCreateContainerRequest(siteUrl: string, driveId: string, filePath: string, fileName: string, createShareLinkType?: ISharingLinkKind, containerPackageInfo?: IContainerPackageInfo | undefined): IRequest; +export function createOdspCreateContainerRequest(siteUrl: string, driveId: string, filePath: string, fileName: string, createShareLinkType?: ISharingLinkKind): IRequest; // @alpha export function createOdspUrl(l: OdspFluidDataStoreLocator): string; diff --git a/packages/drivers/odsp-driver/src/createOdspCreateContainerRequest.ts b/packages/drivers/odsp-driver/src/createOdspCreateContainerRequest.ts index f1bf103e2856..fba2d6096394 100644 --- a/packages/drivers/odsp-driver/src/createOdspCreateContainerRequest.ts +++ b/packages/drivers/odsp-driver/src/createOdspCreateContainerRequest.ts @@ -4,13 +4,10 @@ */ import { IRequest } from "@fluidframework/core-interfaces"; -import { - DriverHeader, - type IContainerPackageInfo, -} from "@fluidframework/driver-definitions/internal"; +import { DriverHeader } from "@fluidframework/driver-definitions/internal"; import { ISharingLinkKind } from "@fluidframework/odsp-driver-definitions/internal"; -import { buildOdspShareLinkReqParams, getContainerPackageName } from "./odspUtils.js"; +import { buildOdspShareLinkReqParams } from "./odspUtils.js"; /** * Create the request object with url and headers for creating a new file on OneDrive Sharepoint @@ -20,7 +17,6 @@ import { buildOdspShareLinkReqParams, getContainerPackageName } from "./odspUtil * @param fileName - name of the new file to be created * @param createShareLinkType - type of sharing link you would like to create for this file. ShareLinkTypes * will be deprecated soon, so for any new implementation please provide createShareLinkType of type ShareLink - * @param containerPackageInfo - container package information which will be used to extract the container package name. * If not given that means that the container package does not have a name. * @legacy * @alpha @@ -31,13 +27,12 @@ export function createOdspCreateContainerRequest( filePath: string, fileName: string, createShareLinkType?: ISharingLinkKind, - containerPackageInfo?: IContainerPackageInfo | undefined, ): IRequest { const shareLinkRequestParams = buildOdspShareLinkReqParams(createShareLinkType); const createNewRequest: IRequest = { url: `${siteUrl}?driveId=${encodeURIComponent(driveId)}&path=${encodeURIComponent( filePath, - )}${containerPackageInfo ? `&containerPackageName=${getContainerPackageName(containerPackageInfo)}` : ""}${shareLinkRequestParams ? `&${shareLinkRequestParams}` : ""}`, + )}${shareLinkRequestParams ? `&${shareLinkRequestParams}` : ""}`, headers: { [DriverHeader.createNew]: { fileName, diff --git a/packages/drivers/odsp-driver/src/test/odspDriverResolverTest.spec.ts b/packages/drivers/odsp-driver/src/test/odspDriverResolverTest.spec.ts index 4a8ee49a52d7..79809f4ddd93 100644 --- a/packages/drivers/odsp-driver/src/test/odspDriverResolverTest.spec.ts +++ b/packages/drivers/odsp-driver/src/test/odspDriverResolverTest.spec.ts @@ -457,20 +457,4 @@ describe("Odsp Driver Resolver", () => { `https://placeholder/placeholder/${resolvedUrl.hashedDocumentId}/` + `${testFilePath}`; assert.strictEqual(resolvedUrl.url, expectedResolvedUrl, "resolved url is wrong"); }); - it("Should create request with containerPackageName and resolve it", async () => { - request = createOdspCreateContainerRequest( - siteUrl, - driveId, - filePath, - fileName, - undefined, - { name: "testContainerPackageName" }, // Container package info variable, - ); - const resolvedUrl = await resolver.resolve(request); - assert.strictEqual( - resolvedUrl.codeHint?.containerPackageName, - "testContainerPackageName", - "containerPackageName should match", - ); - }); });