Skip to content

Every call to a setter results in a call to refreshUrl #21

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.

Sign up for GitHub

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

Jump to bottom
Open
javier-godoy opened this issue Dec 2, 2022 · 1 comment
Open

Every call to a setter results in a call to refreshUrl #21

javier-godoy opened this issue Dec 2, 2022 · 1 comment

Comments

@javier-godoy
Copy link
Member

Every call to a setter results in a call to refreshUrl, which fetches the items again.

RssItemsAddon/src/main/java/com/flowingcode/vaadin/addons/rssitems/RssItems.java

Line 115 in 685c5ab

private void refreshUrl() {

@javier-godoy
Copy link
Member Author

When a property changes, do not refreshUrl immediately. Instead, schedule a beforeClientResponse execution and track what is the currently scheduled execution, so that it can be cancelled when another property is modified during the same request.

@paodb paodb moved this to To Do in Flowing Code Addons May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Flowing Code Addons
Status: To Do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@javier-godoy