Skip to content

Add upload UI and wire the uploader into the grid - #25622

Draft
crazytonyli wants to merge 4 commits into
task/media-v2-upload-enginefrom
task/media-v2-upload-ui
Draft

Add upload UI and wire the uploader into the grid#25622
crazytonyli wants to merge 4 commits into
task/media-v2-upload-enginefrom
task/media-v2-upload-ui

Conversation

@crazytonyli

Copy link
Copy Markdown
Contributor

Note

This PR will be merged after #25621.

@dangermattic

Copy link
Copy Markdown
Collaborator
2 Warnings
⚠️ View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.
⚠️ This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.
1 Message
📖 This PR is still a Draft: some checks will be skipped.

Generated by 🚫 Danger

@wpmobilebot

wpmobilebot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

🤖 Build Failure Analysis

This build has failures. Claude has analyzed them - check the build annotations for details.

@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch from 1e1fa49 to 90df03b Compare June 8, 2026 23:18
@wpmobilebot

wpmobilebot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number34024
VersionPR #25622
Bundle IDorg.wordpress.alpha
Commit5bc6594
Installation URL5t4622h4lmc70
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot

wpmobilebot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor
App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number34007
VersionPR #25622
Bundle IDcom.jetpack.alpha
Commit2afa80c
Installation URL2m1vlndtdk8f0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch from 90df03b to 362fcbe Compare June 11, 2026 00:32
@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch from 362fcbe to 14694c9 Compare June 11, 2026 21:30
@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch from 14694c9 to 5db0706 Compare June 12, 2026 00:50
@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch 2 times, most recently from 80dac4b to 3206ce1 Compare June 16, 2026 03:43
@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch from 3206ce1 to 9e91167 Compare June 16, 2026 08:28
@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch from 9e91167 to 019a63c Compare June 17, 2026 23:51
@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch from 019a63c to 8384343 Compare June 19, 2026 00:20
@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch from 8384343 to 8682318 Compare July 7, 2026 22:09
@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch 2 times, most recently from cb78b3b to 2afa80c Compare August 26, 2026 07:39
Pending and failed entries carry the materialized temp file URL so the
Uploads screen can render a thumbnail. It is nil until materialization
completes and stays nil for non-retryable failures.
The registry caches one MediaUploader per blog so in-flight uploads
survive navigation, and pushes a freshly built policy into a cached
uploader on every vend so changed media settings apply to new enqueues.
The factory derives the policy from the blog's allowed types and the
user's media settings.
Add an add-menu with photo library, camera, file, and app-supplied
external pickers; a banner above the grid summarizing pending and failed
uploads; and an Uploads sheet with per-row cancel, retry, and remove plus
bulk actions. The view model observes the uploader's state publisher and
maps snapshots into row items. The routing vends the site's uploader from
the registry into the hosting controller.
@crazytonyli
crazytonyli force-pushed the task/media-v2-upload-ui branch from 2afa80c to 5bc6594 Compare August 26, 2026 23:49
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.

3 participants