Skip to content

dbeaver/pro#8223 add new events for messages#4110

Merged
serge-rider merged 9 commits intodevelfrom
8223-warning-message
Mar 11, 2026
Merged

dbeaver/pro#8223 add new events for messages#4110
serge-rider merged 9 commits intodevelfrom
8223-warning-message

Conversation

@yagudin10
Copy link
Copy Markdown
Member

No description provided.

@yagudin10 yagudin10 requested a review from HocKu7 February 3, 2026 15:47
@yagudin10 yagudin10 force-pushed the 8223-warning-message branch from 34eb11a to 627326b Compare February 3, 2026 16:11
@yagudin10 yagudin10 force-pushed the 8223-warning-message branch from 627326b to e8d5b8f Compare February 3, 2026 16:13
sergeyteleshev
sergeyteleshev previously approved these changes Feb 6, 2026
}

.alert.error {
background-color: rgba(255, 77, 79, 0.15);
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.

maybe we can use some of the predefined colors in css variables?

cb_database_output_log_updated,
cb_ai_chat_message_chunk @since(version: "25.1.1")
cb_ai_chat_message_error @since(version: "25.1.1")
cb_ai_chat_message @since(version: "25.3.5")
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.

This is super weird that we put all possible events in a single enum.
And put all event structs in core api.
Let's refactor this in 26.0.1.

@serge-rider serge-rider merged commit 1e05ad9 into devel Mar 11, 2026
10 checks passed
@serge-rider serge-rider deleted the 8223-warning-message branch April 1, 2026 09:32
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.

6 participants