add triggers to sync preferences/wallet #3022

Closed
opened 2019-10-11 18:21:32 +02:00 by tzarebczan · 1 comment
tzarebczan commented 2019-10-11 18:21:32 +02:00 (Migrated from github.com)

Besides just syncing at a 5 minute interval, we should also sync after adding/removing a tag/subscription and after creating a channel.

Today on LBRY.tv, you'd lose any tags if you refresh before the interval. On desktop, you'd lose it if you closed the app or refreshed.

When we add other settings, we'll need to trigger it as well.

Besides just syncing at a 5 minute interval, we should also sync after adding/removing a tag/subscription and after creating a channel. Today on LBRY.tv, you'd lose any tags if you refresh before the interval. On desktop, you'd lose it if you closed the app or refreshed. When we add other settings, we'll need to trigger it as well.
kauffj commented 2019-10-11 23:00:05 +02:00 (Migrated from github.com)

@seanyesmunt please see my commentary on https://github.com/lbryio/lbry-react-native/pull/56 before beginning work on this.

After seeing this ticket as well it makes me think that at least logic for this should be shared between desktop and mobile. Both apps want to trigger pushing user state in the same cases (tags changing, channel creation, etc.).

How can this logic be shared and cleanly separated?

@seanyesmunt please see my commentary on https://github.com/lbryio/lbry-react-native/pull/56 before beginning work on this. After seeing this ticket as well it makes me think that at least logic for this should be shared between desktop and mobile. Both apps want to trigger pushing user state in the same cases (tags changing, channel creation, etc.). How can this logic be shared and cleanly separated?
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: LBRYCommunity/lbry-desktop#3022
No description provided.