Skip to content

[POC] Changes to optimise the upload time for acrobat#790

Merged
arugupta1992 merged 6 commits into
stagefrom
directUploadNopoll
May 26, 2026
Merged

[POC] Changes to optimise the upload time for acrobat#790
arugupta1992 merged 6 commits into
stagefrom
directUploadNopoll

Conversation

@arugupta1992
Copy link
Copy Markdown
Collaborator

  • Add your
  • Specific
  • Features or fixes

Resolves: MWPW-NUMBER

Test URLs:

Ruchika4 and others added 5 commits May 22, 2026 16:32
For files at or under 1MB on the word-to-pdf verb, skip the
createAsset → chunkPdf → finalize chain and instead POST the file
directly as multipart to /asset/upload, then call /asset/connector.
Reduces upload round-trips from 4 to 2 for small files.

Also incorporates verb-finalize optimization: verifyContent is
conditional fire-and-forget for verbs in fireAndForgetFinalize.

Points stage environment at unity-dev for testing the new API.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
continueInApp() guards on operations.length before navigating;
directUploadSingleFile never pushed to operations so the guard
always failed and window.location.href was never set.

Also adds 'uploading' analytics event and setIsUploading(true)
to match the standard upload flow sequence.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@arugupta1992 arugupta1992 requested review from shasmish and vipu0303 May 26, 2026 07:09
@aem-code-sync
Copy link
Copy Markdown

aem-code-sync Bot commented May 26, 2026

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

@arugupta1992 arugupta1992 merged commit e5b0d11 into stage May 26, 2026
6 of 8 checks passed
@aem-code-sync aem-code-sync Bot temporarily deployed to directUploadNopoll1 May 28, 2026 10:18 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants