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

The automated message when setuping multiple devices isn't accurate #3889

Closed
narFnarF opened this issue Dec 30, 2022 · 4 comments · Fixed by #4175
Closed

The automated message when setuping multiple devices isn't accurate #3889

narFnarF opened this issue Dec 30, 2022 · 4 comments · Fixed by #4175
Labels
bug Something is not working

Comments

@narFnarF
Copy link

The automated message that appear when setuping multiple devices isn't accurate.

The current message:
It seems you are using multiple devices with Delta Chat. Great!
If you also want to synchronize outgoing messages accross all devices, go to the settings and enable "Send copy to self".

Multiple issues with this message:

  1. The instruction are vague: There's no "Send copy to self" option in Settings. We need to navigate to the Advanced menu.
    • Solution: Rephrase the last sentence. "go to Settings, then Advanced and enable "Send copy to self". "
  2. The setting of sending a copy to self is currently already ON by default. Maybe this whole message is unecessary?
    • Solution: Detect if the setting is already turned ON and avoid displaying these instructions
  3. Typo in "accross". Should be "across".
@Simon-Laux Simon-Laux transferred this issue from deltachat/deltachat-desktop Jan 1, 2023
@Simon-Laux
Copy link
Member

Simon-Laux commented Jan 1, 2023

Transferred the issue to the core repo string is here:

msg.text = Some(
"It seems you are using multiple devices with Delta Chat. Great!\n\n\
If you also want to synchronize outgoing messages accross all devices, \
go to the settings and enable \"Send copy to self\"."
.to_string(),

@Simon-Laux Simon-Laux added the bug Something is not working label Jan 1, 2023
@gerryfrancis
Copy link
Contributor

To add: Messages sent in the past but not to self are not synchronized, I think this should be mentioned in the notes, too.

@Septias
Copy link
Contributor

Septias commented Mar 8, 2023

  1. I think it's hard to unify the instructions to enable send copy to self because every UI can have a different settings menu. But @Hocuri, @r10s and @Simon-Laux know more about this I guess. If alle the UI's are similar in this regard, we could change the wording.

  2. This line should prevent any user that has bcc-self (send message to self) activated to received said message and in fact for me it didn't show up with this workflow:

    1. Create acc on phone & create Backup
    2. Load backup on desktop

Maybe you can recheck or tell me what you did exactly?

@Hocuri
Copy link
Collaborator

Hocuri commented Mar 19, 2023

One problem can't be reproduced, the other one is reeeally easy to fix, I'll do a PR.

@Septias Thanks for digging up this old low-hanging fruit and pining me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants