Skip to content

feat: DownloadAction for the action trigger framework#24432

Open
Artur- wants to merge 1 commit into
mainfrom
download-action
Open

feat: DownloadAction for the action trigger framework#24432
Artur- wants to merge 1 commit into
mainfrom
download-action

Conversation

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 24, 2026

Test Results

 1 432 files  +2   1 432 suites  +2   1h 22m 33s ⏱️ - 2m 11s
10 060 tests +8   9 992 ✅ +8  68 💤 ±0  0 ❌ ±0 
10 532 runs  +8  10 463 ✅ +8  69 💤 ±0  0 ❌ ±0 

Results for commit 0ce0ba3. ± Comparison against base commit b238ec1.

♻️ This comment has been updated with latest results.

@Artur- Artur- marked this pull request as ready for review May 25, 2026 10:05
@Artur- Artur- marked this pull request as draft May 25, 2026 10:07
@Artur- Artur- force-pushed the download-action branch 2 times, most recently from ca4bad3 to ec5d871 Compare May 26, 2026 19:44
@Artur- Artur- marked this pull request as ready for review May 26, 2026 20:00
@Artur- Artur- force-pushed the download-action branch from ec5d871 to 9f00a65 Compare May 26, 2026 20:11
@github-actions github-actions Bot added +0.1.0 and removed +1.0.0 labels May 26, 2026
Starts a file download in the browser when a trigger fires, using a
synthesised <a href download> click inside the gesture. Three flavours:
a static URL (optionally with a filename), a DownloadHandler whose
stream resource is wired through Element.setAttribute (same lifecycle
as Image.setSrc / Anchor.setHref), and an Action.Input<String> for URLs
resolved on the client at fire time. No outcome callback — browsers do
not report download completion.
@Artur- Artur- force-pushed the download-action branch from 816e651 to 0ce0ba3 Compare May 30, 2026 17:03
@sonarqubecloud
Copy link
Copy Markdown

/**
* Body served by the {@code #download-handler} button's
* {@link DownloadHandler}. The IT fetches the recorded URL and asserts this
* exact string, verifying that {@link Element#setAttribute}-driven
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Qualifier or import missing for the Element API link.

// browser's native download handling instead of being intercepted as an
// in-app navigation. Matches Anchor.setHref(DownloadHandler).
a.setAttribute('router-ignore', '');
a.rel = 'noopener';
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rel="noopener" is noop without target="_blank" and can be removed.

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