[GH-ISSUE #3010] [Bug] changing group, field value view not updating #1320

Closed
opened 2026-03-23 20:47:48 +00:00 by mirror · 4 comments
Owner

Originally created by @zoli on GitHub (Jul 15, 2023).
Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/3010

Originally assigned to: @zoli on GitHub.

Bug Description

On changing card group sometimes the card data (field) doesn't get updated immediately.

How to Reproduce

Change a card group then open it see if the shown value is correct and equal to the current card group.

Expected Behavior

The shown field value should get updated and be equal to card group.

Operating System

Linux

AppFlowy Version(s)

0.2.6

Screenshots

No response

Additional Context

This problem first got mentioned in https://github.com/AppFlowy-IO/AppFlowy/pull/2641.

Originally created by @zoli on GitHub (Jul 15, 2023). Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/3010 Originally assigned to: @zoli on GitHub. ### Bug Description On changing card group sometimes the card data (field) doesn't get updated immediately. ### How to Reproduce Change a card group then open it see if the shown value is correct and equal to the current card group. ### Expected Behavior The shown field value should get updated and be equal to card group. ### Operating System Linux ### AppFlowy Version(s) 0.2.6 ### Screenshots _No response_ ### Additional Context This problem first got mentioned in https://github.com/AppFlowy-IO/AppFlowy/pull/2641.
mirror 2026-03-23 20:47:48 +00:00
  • closed this issue
  • added the
    bug
    label
Author
Owner

@zoli commented on GitHub (Jul 15, 2023):

I would like to work and resolve this.

<!-- gh-comment-id:1636883623 --> @zoli commented on GitHub (Jul 15, 2023): I would like to work and resolve this.
Author
Owner

@zoli commented on GitHub (Jul 16, 2023):

This is similar to #2926, move_group_row doesn't fire DidUpdateViewRows it only fires DidUpdateGroupRow. This time I'm not sure, should the frontend listen to DidUpdateGroupRow (updated_rows) aside DidUpdateViewRows or the backend should fire both DidUpdateGroupRow and DidUpdateViewRows?

Also, IMHO the current flowy-database2 is bug-prone to this type of not sending event bugs. It's not enough narrowed to for example only update cell through one channel.
Again for example I'm not sure but it seems possible to use update_cell for moving a row between groups.

@appflowy

<!-- gh-comment-id:1637196128 --> @zoli commented on GitHub (Jul 16, 2023): This is similar to #2926, `move_group_row` doesn't fire `DidUpdateViewRows` it only fires `DidUpdateGroupRow`. This time I'm not sure, should the frontend listen to `DidUpdateGroupRow` (`updated_rows`) aside `DidUpdateViewRows` or the backend should fire both `DidUpdateGroupRow` and `DidUpdateViewRows`? Also, IMHO the current flowy-database2 is bug-prone to this type of not sending event bugs. It's not enough narrowed to for example only update cell through one channel. Again for example I'm not sure but it seems possible to use `update_cell` for moving a row between groups. @appflowy
Author
Owner

@zoli commented on GitHub (Aug 28, 2023):

Just wanted to note this issue still exists.

<!-- gh-comment-id:1696424292 --> @zoli commented on GitHub (Aug 28, 2023): Just wanted to note this issue still exists.
Author
Owner

@zoli commented on GitHub (Oct 30, 2023):

This is almost (I will add some notes later) fixed by https://github.com/AppFlowy-IO/AppFlowy/pull/3730.

<!-- gh-comment-id:1784649710 --> @zoli commented on GitHub (Oct 30, 2023): This is almost (I will add some notes later) fixed by https://github.com/AppFlowy-IO/AppFlowy/pull/3730.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#1320
No description provided.