Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-75344] Fix custom URL name of result action #1970

Merged
merged 4 commits into from
Mar 9, 2025

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Mar 8, 2025

In #1919 a new icon property has been added to the recorder. This accidentally broke the URL mapping of the warning results.

When a custom ID is given, then this ID will be used as URL. Only for empty URLs, the default ID of the label provider will be used.

Additionally, the log messages have been fixed when the ID is used in tool and recorder (see JENKINS-75391).

In #1919
a new icon property has been added to the recorder.
This accidentally broke the URL mapping of the warning results.

When a custom ID is given, then this ID will be used as URL.
Only for empty URLs, the default ID of the label provider will be used.

Additionally, the log messages have been fixed when the ID
is used in tool and recorder (see JENKINS-75391).
@uhafner uhafner added the bug Bugs or performance problems label Mar 8, 2025
uhafner added 3 commits March 9, 2025 11:27
Data-Tables changed the attributes of the visible items
`select` element. Rather than searching by name, now we
select the element by ID prefix.
@uhafner uhafner merged commit 1a10f56 into main Mar 9, 2025
33 of 41 checks passed
@uhafner uhafner deleted the JENKINS-75344-fix-custom-url branch March 9, 2025 11:57
uhafner added a commit that referenced this pull request Mar 10, 2025
Use the ID of the result action when creating a project action.
This is a followup to
#1970
uhafner added a commit that referenced this pull request Mar 10, 2025
Use the ID of the result action when creating a project action.
This is a followup to
#1970
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs or performance problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant