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

[16.0][ADD] base_live_update #849

Open
wants to merge 2 commits into
base: 16.0
Choose a base branch
from
Open

Conversation

hbrunn
Copy link
Member

@hbrunn hbrunn commented Mar 30, 2024

This module provides a framework other modules can use to update the
client side UI when records of some model have changed.

To test this on runboat, allow notifications, go to a kanban view of partners, in another window/computer/user, change the name of some partner, and see this change reflected in the kanban view. Or group the view ie by company, and drag some partner into another company.

Or install whatever module you're interested in and add the model in question to Settings/Technical/Live update notifications.

@StefanRijnhart
Copy link
Member

Workes very nicely already on the partner kanban view! For my purposes the kanban card moving between groups is the killer purpose here. Maybe that bit does not need to be in a separate module?

@hbrunn
Copy link
Member Author

hbrunn commented Apr 1, 2024

indeed, I'm more and more of a mind to do all of that in the base module, as we depend on web already anyways

@hbrunn hbrunn changed the title [RFC][16.0][ADD] base_live_update (unfinished) [16.0][ADD] base_live_update Apr 8, 2024
@hbrunn
Copy link
Member Author

hbrunn commented Apr 8, 2024

@StefanRijnhart killer purpose implemented. Very inefficiently (see TODOs), which defeats the purpose of this module to some degree, but I think this is good enough for version 1 and can be iterated on.

Copy link

@antoniocanovas antoniocanovas left a comment

Choose a reason for hiding this comment

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

Functional review OK !!

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.

None yet

3 participants