Simplify `Settings` (#285)
Closed
Andrzej Wójcik (Tigase) opened 3 years ago

As it was discussed on GitHub our default settings (https://github.com/tigase/siskin-im/issues/92) and a way to suggest enabling features are suboptimal. It would be good to move to enable Push to the global scope (so that user would enable it once for all accounts which support these features). It would be also good to replace MAM options:

  • enabled
  • synchronization enabled
  • synchronization range (message from X hours/days)

with by having an on/off feature on the account level and only asking the user during adding the account how many messages he wishes to initially fetch.

Having MAM enabled would automatically trigger MAM synchronization from the last message available in the local storage.

Additionally, we should implement fetch more feature, to automatically try to fetch older messages if we are at the begining of our local storage. (some flag marking that there is nothing more to fetch if we know that we are at the end of the archive for this conversation would be nice).

Andrzej Wójcik (Tigase) commented 3 years ago

I've applied described changes and make sure to disable push notifications for an account that is about to be disabled in the settings.

issue 1 of 1
Type
Task
Priority
Normal
Assignee
Version
7.0
Issue Votes (0)
Watchers (0)
Reference
tigase/_clients/siskin-im#285
Please wait...
Page is in error, reload to recover