Skip to content

LKE-14893: Expose plugin actions#35

Draft
gmasclet wants to merge 1 commit into
developfrom
LKE-14893-expose-plugin-actions
Draft

LKE-14893: Expose plugin actions#35
gmasclet wants to merge 1 commit into
developfrom
LKE-14893-expose-plugin-actions

Conversation

@gmasclet
Copy link
Copy Markdown
Contributor

@gmasclet gmasclet commented May 20, 2026

Jira ref.: https://linkurious.atlassian.net/browse/LKE-14893

Description

🚧 Work in progress 🚧

Demonstrate how https://github.com/Linkurious/linkurious-server/pull/6220 can be used.

This PR is not complete: here, the plugin is still creating regular custom actions. We should remove that logic if the IPC client is defined.

@gmasclet gmasclet force-pushed the LKE-14893-expose-plugin-actions branch from 4255f0d to 06d53bb Compare May 20, 2026 11:55
isAdmin: boolean;
}

export class API extends WithLogger {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I'm missing something, but I feel the API is an empty shell which doesn't bring actual value.

import {Logger, WithLogger} from '../services/logger';

interface PluginOptions extends PluginRouteOptions<MyPluginConfig> {
ipcClient?: IpcClient;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

defined as optional, so that the plugin stays compatible with former LKE hosts.

actions: [
{
name: 'Configure integrations',
urlTemplate: `{{baseURL}}plugins/${basePath}/`,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We may also consider providing a template that is relative to the plugin base path (and let the server resolve the absolute URL if needs be).

Suggested change
urlTemplate: `{{baseURL}}plugins/${basePath}/`,
urlTemplate: '/',

@gmasclet gmasclet requested a review from davidrapin May 20, 2026 12:38
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