Skip to content

Send 'qute/dataModelChanged' notification with debounce mode#1099

Merged
angelozerr merged 1 commit into
redhat-developer:masterfrom
angelozerr:qute_listener_debounce
Jan 28, 2026
Merged

Send 'qute/dataModelChanged' notification with debounce mode#1099
angelozerr merged 1 commit into
redhat-developer:masterfrom
angelozerr:qute_listener_debounce

Conversation

@angelozerr
Copy link
Copy Markdown
Contributor

@angelozerr angelozerr commented Jan 27, 2026

Send 'qute/dataModelChanged' notification with debounce mode

Same idea than eclipse-lsp4mp/lsp4mp#541

@angelozerr angelozerr self-assigned this Jan 27, 2026
@angelozerr angelozerr added this to the 0.23.1 milestone Jan 27, 2026
@angelozerr angelozerr moved this to In Progress in Java Tooling Jan 27, 2026
@angelozerr angelozerr requested a review from datho7561 January 27, 2026 19:37
Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

It's working well. However, the notification as it appears in the LSP logs is qute/dataModelChanged but in some parts of the codebase is refered to as qute/javaDataModelChanged

@angelozerr angelozerr force-pushed the qute_listener_debounce branch from b3f3123 to 5753939 Compare January 27, 2026 20:59
Signed-off-by: azerr <azerr@redhat.com>
@angelozerr angelozerr force-pushed the qute_listener_debounce branch from 5753939 to 5f777ab Compare January 27, 2026 21:36
@angelozerr
Copy link
Copy Markdown
Contributor Author

It's working well. However, the notification as it appears in the LSP logs is qute/dataModelChanged but in some parts of the codebase is refered to as qute/javaDataModelChanged

qute/dataModelChanged is the right name, I have tried to fix that.

Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks, Angelo!

@angelozerr angelozerr merged commit 5e9db28 into redhat-developer:master Jan 28, 2026
2 of 4 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Java Tooling Jan 28, 2026
@angelozerr
Copy link
Copy Markdown
Contributor Author

Thanks @datho7561 for your review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants