Skip to content

dbeaver/pro#9999 api for ai engine icon - #4518

Open
yagudin10 wants to merge 6 commits into
develfrom
9999-ai-engine-icon
Open

dbeaver/pro#9999 api for ai engine icon#4518
yagudin10 wants to merge 6 commits into
develfrom
9999-ai-engine-icon

Conversation

@yagudin10

Copy link
Copy Markdown
Member

No description provided.

@yagudin10
yagudin10 requested a review from HocKu7 July 31, 2026 08:53
@codacy-production

codacy-production Bot commented Jul 31, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 6 complexity · 0 duplication

Metric Results
Complexity 6
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

sergeyteleshev
sergeyteleshev previously approved these changes Aug 4, 2026
);
}

return <span title={title}>{title}</span>;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lets show and icon and the engine name. users cannot know each engine by only icon. right now they have to hover too see the name and we can easily avoid this action since there is enough space in the table

import { AiEnginesResource } from './AiEnginesResource.js';

export default ModuleRegistry.add({
name: '@cloudbeaver/plugin-ai',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

plugin-ai is too common name. in the future we may want to place everything to it instead of decomposing ai-features to the plugins, I guess

plugin-ai-engines?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We have plugin-ai-administration, so it seems like plugin-ai would be a suitable public plugin name here, at least for now

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.

4 participants