Deltachat-android: Add preference / settings functionality

Created on 29 Aug 2018  路  16Comments  路  Source: deltachat/deltachat-android

After login the user is unable to open the settings. The app crashes as we have no address object, which is needed. To avoid a crash and provide the needed functionality we should adjust the ProfilePreference.java and other needed classes to display DeltaChat releated user information.

  • [x] Avoid crash on entering settings screen
  • [x] Avoid crash on entering account settings screen and enable the adjustment of account settings
  • [x] Allow enabling / disabling of app locking, see #42 for more information
  • [x] Add security check to backup and key export
  • [x] Remove Settings / Notifications / Calls category

All 16 comments

_This is probably a good task for myself to get in touch with the Dc* interfaces._

First iteration added (https://github.com/deltachat/deltachat-android-ii/commit/1b1d20f53be659c4e912ae1ae82c5dff292e847b), which avoids the initial crash and gets the first config values via the core.

I will now adjust the edit profile part, which is currently also crashing, as it references the address object mentioned in https://github.com/deltachat/deltachat-android-ii/issues/17.

Added https://github.com/deltachat/deltachat-android-ii/commit/75971f7320d970d95afcbe655a80e2447b964792 to allow the user to change the name and the avatar image within the settings. The settings are applied and used now.

great :)

my suggestions:

  • i would open a local Alert instead of an URL when clicking on "Who can see this information?"
  • i would add a "back arrow" button to the action bar, esp. as the normal. the arrow can have the same effect as "set later".
  • maybe we can add an additional button "password and account settings" below "set later" and open the RegistrationActivity from there (to re-configure the account)

Commit: https://github.com/deltachat/deltachat-android-ii/commit/08a4e63ea411d091e95777feaca8c1ca408947e5

Cleaned up the "Privacy" setting fragment

  • removed REGISTRATION_LOCK_PREF as we have no phone number registration
  • removed ALWAYS_RELAY_CALLS_PREF as we don't have calls and even if we would support calls we don't want to relay them using specific servers

Changed / added the suggested adjustments:

great :)

my suggestions:

* i would open a local Alert instead of an URL when clicking on "Who can see this information?"

* i would add a "back arrow" button to the action bar, esp. as the normal. the arrow can have the same effect as "set later".

* maybe we can add an additional button "password and account settings" below "set later" and open the RegistrationActivity from there (to re-configure the account)

The mentioned optimizations were implemented. For the "who can see this" stuff I wrote a short sentence which can get adjusted later if needed.

Commit: https://github.com/deltachat/deltachat-android-ii/commit/8ad1652cc6e4f7c98342a5132cdb2c333e32f6df

  • added an information toast if the "Screen security" setting is changed. As the user needs to restart the app before this setting will be active

cool :)

i think the credentials in the RegistrationActivity should be pre-filled with the current settings - incl. the password so that the user can click reconfigure without doing changes (not that this is always useful, however, people will do so :)

Will gather potential other tasks in the preferences area tomorrow and will add them as sub tasks to this ticket.

@r10s I added "Remove 'Settings / Notifications / Calls' category" as task. Is this a good idea for now or should it stay in the settings?

@Boehrsi no, we can remove them.
i also cleaned up the chats activity yesterday.

Added the security check to the backup and key export.

great :)

Removed Settings / Notifications / Calls category. I would say the main part of the settings is now working as intended and we should close this issue for now. Other problems with the preferences could be tackled in newly created tickets. What's you opinion about this @r10s ?

also removed "contact-joined-deltachat", see #67, closing this now :)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

csb0730 picture csb0730  路  4Comments

adbenitez picture adbenitez  路  4Comments

gerryfrancis picture gerryfrancis  路  4Comments

gerroon picture gerroon  路  3Comments

angelo-fuchs picture angelo-fuchs  路  4Comments