Skip to content

feat: user-friendly display of pull event handler name#967

Merged
isc-pbarton merged 1 commit into
mainfrom
display-pull-event-handler-name
May 11, 2026
Merged

feat: user-friendly display of pull event handler name#967
isc-pbarton merged 1 commit into
mainfrom
display-pull-event-handler-name

Conversation

@isc-pbarton
Copy link
Copy Markdown
Collaborator

Description

Resolves #908

The "Pull Event Handler" dropdown in the Embedded Git settings page now displays the user-friendly name for options as specified by the NAME parameter instead of the class name. Hover text on the options displays the class name and the description.

Testing

docker compose up --build -d

Visited http://localhost:52774/isc/studio/usertemplates/gitsourcecontrol/gitprojectsettings.csp and tested viewing and changing the pull event handler input. Tested saving the form and confirmed it persisted.
image

Checklist

  • This branch has the latest changes from the main branch rebased or merged.
  • Web UI has been built (any changes in git-webui/src have matching changes in git-webui/release)
  • CHANGELOG.md entry added if appropriate.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.10%. Comparing base (6438970) to head (4ee2d51).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #967   +/-   ##
=======================================
  Coverage   49.10%   49.10%           
=======================================
  Files          23       23           
  Lines        3287     3287           
=======================================
  Hits         1614     1614           
  Misses       1673     1673           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@isc-pbarton isc-pbarton merged commit c0d7b72 into main May 11, 2026
2 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.

Web UI: Settings display improvement suggestion

3 participants