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.

Sign up for GitHub

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

Jump to bottom

[Bug]: Settings / Account menu - Make clear to confirm changes before leaving #745

Open
3 tasks done
Echolon opened this issue Nov 7, 2021 · 3 comments · May be fixed by #1283
Open
3 tasks done

[Bug]: Settings / Account menu - Make clear to confirm changes before leaving #745

Echolon opened this issue Nov 7, 2021 · 3 comments · May be fixed by #1283
Assignees
@thevaidik
Labels
🎨 UI Related to visual appreance of the app (User Interface) 📱 iOS Related to iOS / iPhone / iPad Minor Due to limited ressources, this issue will be treated with a minor priority. 🚫 Bug / Stability Releated directly to bugs or instability issues 🎃 Short GSOC Teaser A short task to start contributing with 🦸 Help wanted If you are motivated, this is a good start!

Comments

@Echolon
Copy link
Contributor

Echolon commented Nov 7, 2021

What Monal Release channel are you using?

AppStore

iOS system version

14.8

iOS Monal version

5.0.1

macOS system version

No response

macOS Monal version

No response

Used XMPP server (domain)

yax.im

Which XMPP-Server software are you using?

Prosody

XMPP Server Software Version

No response

How many accounts are you using in Monal?

2

What happened?

  1. Go to Settings
  2. Go to your account
  3. Deactivate the account toggle
  4. Leave the menu
  5. Nothing changed

--> Monal should ask you if you want to confirm your changes - or just do it on change

Anything else?

  1. Monal should tell the 'offline' user why they cannot write messages anymore

Related Issues

XEP-Check

  • I have checked that at least XEP-198, XEP-0280, XEP-0352, XEP-0357 and XEP-0163 are activated on my server and shown as 'green' under Settings --> Account--> (i) in advanced settings

Notifications-Menu

  • I have checked that all checkboxes are ticked under Settings --> Notifications
@Echolon Echolon added 🚫 Bug / Stability Releated directly to bugs or instability issues 📱 iOS Related to iOS / iPhone / iPad 🎨 UI Related to visual appreance of the app (User Interface) Minor Due to limited ressources, this issue will be treated with a minor priority. labels Nov 7, 2021
@tmolitor-stud-tu tmolitor-stud-tu added the 🦸 Help wanted If you are motivated, this is a good start! label Jan 15, 2023
@lissine0 lissine0 mentioned this issue Oct 4, 2023
Closed
2 tasks
@tmolitor-stud-tu tmolitor-stud-tu added the 🎃 Short GSOC Teaser A short task to start contributing with label Mar 3, 2024
@thevaidik
Copy link
Contributor

Hey, i am working on it, can i be assigned

@tmolitor-stud-tu
Copy link
Member

@thevaidik are you still working on this issue or did you switch to another one?

thevaidik added a commit to thevaidik/Monal-dev that referenced this issue Mar 31, 2024
@thevaidik
autosave fixes monal-im#745
59d8516
thevaidik added a commit to thevaidik/Monal-dev that referenced this issue Mar 31, 2024
@thevaidik
autosave fixes monal-im#745
f4bbb54
@guodong323
Copy link
Contributor

Can I work for it?

thevaidik added a commit to thevaidik/Monal-dev that referenced this issue Apr 4, 2024
@thevaidik
autosave fixes monal-im#745
dff7a91
thevaidik added a commit to thevaidik/Monal-dev that referenced this issue Apr 25, 2024
@thevaidik
autosave fixes monal-im#745
4c7b632
@lissine0 lissine0 linked a pull request Nov 2, 2024 that will close this issue
Draft
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thevaidik thevaidik

Labels
🎨 UI Related to visual appreance of the app (User Interface) 📱 iOS Related to iOS / iPhone / iPad Minor Due to limited ressources, this issue will be treated with a minor priority. 🚫 Bug / Stability Releated directly to bugs or instability issues 🎃 Short GSOC Teaser A short task to start contributing with 🦸 Help wanted If you are motivated, this is a good start!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Draft]: Rewrite the Account Settings view in SwiftUI lissine0/Monal
4 participants
@tmolitor-stud-tu @Echolon @guodong323 @thevaidik