Skip to content

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

Merged
arugupta1992 merged 7 commits into
stagefrom
directUploadNopoll1
May 28, 2026
Merged

[POC] Changes to optimise the upload time for acrobat#795
arugupta1992 merged 7 commits into
stagefrom
directUploadNopoll1

Conversation

@arugupta1992
Copy link
Copy Markdown
Collaborator

  • Add your
  • Specific
  • Features or fixes

Resolves: MWPW-NUMBER

Test URLs:

Ruchika4 and others added 6 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>
@aem-code-sync
Copy link
Copy Markdown

aem-code-sync Bot commented May 28, 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 changed the title Direct upload nopoll1 [POC] Changes to optimise the upload time for acrobat May 28, 2026
@arugupta1992 arugupta1992 merged commit 7f1e0c0 into stage May 28, 2026
5 of 8 checks passed
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.

2 participants