Introduce experimental API for adding actions to experimental Run UI#11204
Introduce experimental API for adding actions to experimental Run UI#11204janfaracik wants to merge 41 commits intojenkinsci:masterfrom
Conversation
3c16200 to
db697ec
Compare
| return null; | ||
| } | ||
|
|
||
| default Group getGroup() { |
There was a problem hiding this comment.
Should this be part of Action? Or into its own interface?
| return null; | ||
| } | ||
|
|
||
| default Group getGroup() { |
There was a problem hiding this comment.
Is this the best way of grouping/ordering items?
| @@ -106,102 +111,56 @@ public void generateResponse(StaplerRequest2 req, StaplerResponse2 rsp, Object o | |||
| } | |||
|
|
|||
| public ContextMenu add(String url, String text) { | |||
There was a problem hiding this comment.
This file needs validating (maybe a clean up if possible without breaking plugins...)
core/src/main/resources/hudson/model/AbstractBuild/sidepanel.jelly
Outdated
Show resolved
Hide resolved
|
Hi @janfaracik, I tested the build page with Please let me know if you’d like help with:
Happy to assist. |
Any testing is appreciated |
timja
left a comment
There was a problem hiding this comment.
I've pushed a commit to mark the API as beta.
I think we're good to go here, more feedback is welcome, happy to change the design.

This PR introduces an experimental API for adding actions to the experimental Run UI - actions are visible in the app bar and its overflow menu.
WIP screenshot
I've been wanting to get this done for a long (long) time now, the intention is to provide a simple to use API for developers to add simple and complex actions to our Actionable objects.
An action can be as simple as taking the user to a new page, or it can be as complex as executing arbitrary JS or displays dialogs and modals.
Testing done
Proposed changelog entries
Proposed changelog category
/label web-ui,rfe
Proposed upgrade guidelines
N/A
Submitter checklist
@Restrictedor have@since TODOJavadocs, as appropriate.@Deprecated(since = "TODO")or@Deprecated(forRemoval = true, since = "TODO"), if applicable.evalto ease future introduction of Content Security Policy (CSP) directives (see documentation).Desired reviewers
@jenkinsci/sig-ux @daniel-beck
Before the changes are marked as
ready-for-merge:Maintainer checklist
upgrade-guide-neededlabel is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidateto be considered (see query).