Skip to content

Add tool owner-initiated update workflow trigger#77

Draft
Copilot wants to merge 5 commits intomainfrom
copilot/add-tool-update-invocation
Draft

Add tool owner-initiated update workflow trigger#77
Copilot wants to merge 5 commits intomainfrom
copilot/add-tool-update-invocation

Conversation

Copy link
Contributor

Copilot AI commented Mar 22, 2026

  • Create new API endpoint POST /api/tools/[id]/trigger-update
  • Update /api/dashboard to include packagename and version fields in tools response
  • Update dashboard page UI to add "Trigger Update" button in "My Tools" view
  • Show custom validation error modal when package validation fails
  • Disable ALL tools' "Trigger Update" buttons while any update is in progress (no parallel execution)
  • Add "Version" column to "My Tools" view (monospace v{version} badge, -- if not set)
  • Move Trigger Update, Deprecate, Delete into a "More ▾" dropdown in "My Tools" view (keep "View" as standalone link)
  • Fix dropdown z-index issue: elevated stacking context on active row + removed overflow-hidden from card wrapper

📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

@vercel
Copy link

vercel bot commented Mar 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
web Ready Ready Preview, Comment Mar 24, 2026 4:08am

Copilot AI changed the title [WIP] Add ability for tool owner to invoke tool update process Add tool owner-initiated update workflow trigger Mar 22, 2026
Copilot AI requested a review from Power-Maverick March 22, 2026 04:28
@Power-Maverick
Copy link
Contributor

@copilot when the package validation fails then show a custom modal providing all the validation errors; do not fail the validation on first but collate all validation errors and warnings and show them to the user. Make sure to mention the name of the package. Also when a validation is invoked for a tool; disable invoking the update for any other tool - no parallel execution allowed for tool update trigger

… triggers

Co-authored-by: Power-Maverick <36135520+Power-Maverick@users.noreply.github.com>
Agent-Logs-Url: https://github.com/PowerPlatformToolBox/pptb-web/sessions/59fc288e-2dfa-437e-b12c-c96334c5d6c4
Copy link
Contributor Author

Copilot AI commented Mar 24, 2026

@copilot when the package validation fails then show a custom modal providing all the validation errors; do not fail the validation on first but collate all validation errors and war...

Done in f3aa067. Here's what changed:

Validation modal — instead of an alert(), a modal now appears showing the package name, all collated errors (red), and all warnings (amber). Dismissible via Close button, backdrop click, or Escape key.

No parallel execution — changed disabled={triggeringUpdateForToolId === tool.id || ...} to disabled={triggeringUpdateForToolId !== null || ...}, so ALL "Trigger Update" buttons are disabled while any one tool's update workflow is running.

Dashboard preview

@Power-Maverick
Copy link
Contributor

@copilot on the My Tools view add the current version of the tool, keep action View as is but move all other options under More like Trigger Update, Deprecate, Delete

Copy link
Contributor Author

Copilot AI commented Mar 24, 2026

@copilot on the My Tools view add the current version of the tool, keep action View as is but move all other options under More like Trigger Update, Deprecate, Delete

Done in d7645fd.

  • Version column added to "My Tools" — shows v{version} as a monospace badge, or -- if not set.
  • View remains a standalone link.
  • Trigger Update, Deprecate, and Delete are now under a More ▾ dropdown that closes on outside click or Escape.

My Tools with Version column and More dropdown

@Power-Maverick
Copy link
Contributor

@copilot there seems to be z-index issue. see the screenshot
image

Co-authored-by: Power-Maverick <36135520+Power-Maverick@users.noreply.github.com>
Agent-Logs-Url: https://github.com/PowerPlatformToolBox/pptb-web/sessions/35f1a7df-135a-4672-afe0-b9731c3dd784
Copy link
Contributor Author

Copilot AI commented Mar 24, 2026

@copilot there seems to be z-index issue. see the screenshot
<img width="269" height="297" alt="image" src="https://github.com/user-attachments/assets/f2c4a79f-202a-421a-bb21-5a2d68...

Fixed in fba4efb. Two changes:

  1. overflow-visible on the card wrapper — previously overflow-hidden was clipping any dropdown that extended past the card's bounds.
  2. relative z-10 on the active <tr> — when a menu is open, that row now creates a stacking context that sits above subsequent rows, preventing later rows from painting over the dropdown.

The "More ∨" text from the row below no longer overlaps the open menu.

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