Skip to content

feat(dev) - #46

Open
roncodes wants to merge 1 commit into
mainfrom
feature/ai-resource-actions
Open

roncodes wants to merge 1 commit into
mainfrom
feature/ai-resource-actions

Conversation

@roncodes

Copy link
Copy Markdown
Member

Why

The API key and webhook dialogs lived inside the api-keys and webhooks index controllers, so nothing outside those routes could open them. The Fleetbase AI extension needs to open them from the AI panel on any page, and query-param deep links were rejected as the mechanism.

What

  • New api-key-actions service (createApiKey, editApiKey, expiration options, test-mode handling) and webhook-actions service (loadWebhookOptions for events, versions and credentials, plus create, edit and delete), both extending ember-core's ResourceActionService.
  • The index controllers now delegate to those services; the methods were moved across as they were, so behavior is unchanged.
  • App re-exports so universe.getServiceFromEngine('@fleetbase/dev-engine', 'api-key-actions') can reach them.

Testing

Lint is clean and the build succeeds. The dialogs have not been clicked through in a browser yet — worth doing for New API Key, editing a key, and New Webhook before merge.

Related

Part of the Fleetbase AI audit; iam-engine gets the same treatment for users, groups, roles and policies.

Move the create, edit and delete dialogs out of the index controllers into
`api-key-actions` and `webhook-actions` services extending ember-core's
`ResourceActionService`. The controllers now delegate to them, so behavior is
unchanged.

This lets other engines (the AI extension in particular) open the Developers
dialogs through `universe.getServiceFromEngine` instead of query-param deep links.
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.

1 participant