Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(entities-plugin): add streaming custom plugin support #1896

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nekolab
Copy link
Contributor

@nekolab nekolab commented Jan 15, 2025

Changes include:

  1. Remove disableCustomPlugins
  2. A new prop customPluginSupport is added to indicate the which type of custom plugin should be created for the CP
  3. The plugin grid now renders schema-only and streaming custom plugins correctly.
  4. getCustomEditRoute now provides type: 'schema' | 'streaming' for custom plugin type indication
  5. Text and sandbox change

JIRA: KM-895

@nekolab nekolab self-assigned this Jan 15, 2025
@nekolab nekolab force-pushed the feat/plugin-select-streaming branch 4 times, most recently from cfd823b to 316c7e4 Compare January 17, 2025 06:04
@nekolab nekolab marked this pull request as ready for review January 20, 2025 02:43
@nekolab nekolab requested review from a team as code owners January 20, 2025 02:43

- type: `boolean`
- type: `false | 'disabled' | 'schema' | 'streaming'`
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you think it will be better to use 'hidden' or 'none' instead of keeping the boolean type here?

@nekolab nekolab force-pushed the feat/plugin-select-streaming branch 3 times, most recently from cb7d3e8 to 27c8d62 Compare January 20, 2025 07:41
Changes include:
1. Remove `disableCustomPlugins`
2. A new prop `customPluginSupport` is added to indicate the
   which type of custom plugin should be created for the CP
3. The plugin grid now renders schema-only and streaming
   custom plugins correctly.
4. `getCustomEditRoute` now provides `type: 'schema' | 'streaming'`
   for custom plugin type indication
5. Text and sandbox change

JIRA: KM-895

Remove disableCustomPlugins and use customPlugins instead
The customPlugins indicates custom plugin support and which type of custom plugins should be created for the control plane
getCustomEditRoute now provides type: 'streaming' | 'non-streaming' for custom plugin type indication
Text and sandbox change
JIRA: KM-895
@nekolab nekolab force-pushed the feat/plugin-select-streaming branch from 27c8d62 to 30996c1 Compare January 20, 2025 08:42
@kongponents-bot
Copy link
Collaborator

Preview components from this PR in consuming application

In consuming application project install preview versions of shared packages generated by this PR:

@kong-ui-public/entities-plugins@pr-1896

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.

3 participants