Skip to content

feat: add Registry field to workflow details header#417

Merged
anirudhwarrier merged 3 commits into
mainfrom
DEVSVCS-4926/ux-improve
May 7, 2026
Merged

feat: add Registry field to workflow details header#417
anirudhwarrier merged 3 commits into
mainfrom
DEVSVCS-4926/ux-improve

Conversation

@anirudhwarrier
Copy link
Copy Markdown
Collaborator

@anirudhwarrier anirudhwarrier commented May 7, 2026

DEVSVCS-4926

This pull request refactors the workflow commands (activate, delete, deploy, and pause) to use a shared function for displaying workflow details, improving code reuse and consistency across commands. The display logic is now centralized in a new DisplayWorkflowDetails function under cmd/workflow/common. Additionally, the registry ID for on-chain registries is now more descriptive.

Improvements

  • Introduced a new DisplayWorkflowDetails function in cmd/workflow/common/display_workflow_details.go to standardize and centralize the display of workflow details across commands. This function also annotates off-chain owner addresses as "(derived)".
  • Add Registry field to display - displays ID of the registry selected.
  • Updated activate.go, delete.go, deploy.go, and pause.go to use the new DisplayWorkflowDetails function instead of their own local displayWorkflowDetails methods. Removed the now-redundant local display methods from each command. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]

Registry ID improvement:

  • Changed the default registry ID for on-chain registries to include the chain name (e.g., onchain:<chainName>) for better clarity in logs and UI.

@anirudhwarrier anirudhwarrier changed the title Devsvcs 4926/ux improve feat: add Registry field to workflow details header May 7, 2026
@anirudhwarrier anirudhwarrier marked this pull request as ready for review May 7, 2026 11:52
@anirudhwarrier anirudhwarrier requested a review from a team as a code owner May 7, 2026 11:52
@anirudhwarrier anirudhwarrier requested a review from ejacquier May 7, 2026 11:52
@anirudhwarrier anirudhwarrier requested review from tarcisiozf and removed request for ibrajer May 7, 2026 11:52
@anirudhwarrier anirudhwarrier self-assigned this May 7, 2026
@anirudhwarrier anirudhwarrier added this pull request to the merge queue May 7, 2026
Merged via the queue into main with commit 276adf6 May 7, 2026
26 checks passed
@anirudhwarrier anirudhwarrier deleted the DEVSVCS-4926/ux-improve branch May 7, 2026 12:42
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.

2 participants