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

[Bug]: Adding and removing additional e-mail adress #31164

Closed
4 of 8 tasks
Valdnet opened this issue Feb 14, 2022 · 6 comments · Fixed by #31345 or #31347
Closed
4 of 8 tasks

[Bug]: Adding and removing additional e-mail adress #31164

Valdnet opened this issue Feb 14, 2022 · 6 comments · Fixed by #31345 or #31347
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: settings

Comments

@Valdnet
Copy link
Contributor

Valdnet commented Feb 14, 2022

⚠️ This issue respects the following points: ⚠️

  • This is a bug, not a question or a configuration/webserver/proxy issue.
  • This issue is not already reported on Github (I've searched it).
  • Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
  • I agree to follow Nextcloud's Code of Conduct.

Bug description

  1. When deleting the secondary email address added first, the edit menu for the second email address is not shown. To delete the second e-mail address you need a page refresh.

Screenshot:

  • Adding two email addresses:

obraz

  • Gray inactive menu button for editing the email address added as the second:

obraz

  • The email address menu is only active after refreshing the page:

obraz

  1. When adding an additional email address, the description "Primary email for password reset and notifications" appears. In my opinion, it is unnecessarily duplicated.

Screenshot:

obraz

Steps to reproduce

  1. Select "Personal info".
  2. Add an additional email address.
  3. Add a second additional email address.
  4. Delete the first additional email address.
  5. Try to delete the second email address.

Expected behavior

  1. While deleting email addresses, every email address editing menu should be active, no matter which one you delete first.
  2. Deletion of duplicate email description "Primary email for password reset and notifications" when adding an additional email address.

@blizzz Can you fix it?

Installation method

Manual installation

Operating system

No response

PHP engine version

PHP 7.3

Web server

No response

Database engine version

MySQL

Is this bug present after an update or on a fresh install?

No response

Are you using the Nextcloud Server Encryption module?

No response

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

No response

List of activated Apps

Accessibility 1.9.0
Activity 2.15.0
Circles 23.0.1
Collaborative tags 1.13.0
Comments 1.13.0
Contacts Interaction 1.4.0
Dashboard 7.3.0
Deleted files 1.13.0
Federation 1.13.0
File sharing 1.15.0
First run wizard 2.12.0
Gestion 2.0.4
Log Reader 2.8.0
Monitoring 1.13.0
Nextcloud announcements 1.12.0
Notes 4.3.0
Notifications 2.11.1
Password policy 1.13.0
PDF viewer 2.4.0
Photos 1.5.0
Privacy 1.7.0
Recommendations 1.2.0
Right click 1.2.0
Share by mail 1.13.0
Support 1.6.0
Text 3.4.0
Theming 1.14.0
Update notification 1.13.0
Usage survey 1.11.0
User status 1.3.1
Versions 1.16.0
Video player 1.12.0
Weather status 1.3.0
Auditing / Logging 1.13.0
Default encryption module 2.11.0
External storage support 1.15.0
LDAP user and group backend 1.13.1

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

@Valdnet Valdnet added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap feature: settings labels Feb 14, 2022
@blizzz
Copy link
Member

blizzz commented Feb 14, 2022

cc @Pytal

@Pytal
Copy link
Member

Pytal commented Feb 24, 2022

Fixed in #31345

@Pytal
Copy link
Member

Pytal commented Feb 24, 2022

Regarding expected behavior

  1. Deletion of duplicate email description "Primary email for password reset and notifications" when adding an additional email address.

I believe this is working as intended @blizzz ?

@blizzz
Copy link
Member

blizzz commented Feb 24, 2022

Regarding expected behavior

  1. Deletion of duplicate email description "Primary email for password reset and notifications" when adding an additional email address.

I believe this is working as intended @blizzz ?

I think he means the duplication of the label as seen in the screenshot

@Pytal
Copy link
Member

Pytal commented Feb 25, 2022

I think he means the duplication of the label as seen in the screenshot

Roger, fixed in #31347

@Pytal
Copy link
Member

Pytal commented Mar 2, 2022

Reopening pending #31345

@Pytal Pytal reopened this Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug feature: settings
Projects
None yet
3 participants