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

Cockpit: new email address view is inconsistent #6739

Closed
federicoballarini opened this issue Mar 28, 2023 · 5 comments
Closed

Cockpit: new email address view is inconsistent #6739

federicoballarini opened this issue Mar 28, 2023 · 5 comments
Labels
bug A defect of the software verified All test cases were verified successfully

Comments

@federicoballarini
Copy link
Member

federicoballarini commented Mar 28, 2023 •
edited

Steps to reproduce

  • Access Cockpit interface and open Email > Addresses page
  • Add first address, selecting the domain:

image

  • Save and add another address without reloading the page

Expected behavior

Domains selectpicker is resetted and no domain is selected

Actual behavior

Domains selectpicker remains selected on the last used domain but if you save, without re-selecting it, the addrress is created as wildcard.

image

Components

nethserver-mail
@federicoballarini federicoballarini added the bug A defect of the software label Mar 28, 2023
federicoballarini added a commit to federicoballarini/nethserver-mail that referenced this issue Mar 28, 2023
@federicoballarini
ui. refresh selectpicker on openCreateAddress
c3fd1c3 
Refresh openCreateAddress selectpicker to avoid inconsistent view after adding first address.
Refer to NethServer/dev#6739
@federicoballarini federicoballarini mentioned this issue Mar 28, 2023
Merged
edospadoni pushed a commit to NethServer/nethserver-mail that referenced this issue Mar 29, 2023
@federicoballarini
ui. refresh selectpicker on openCreateAddress (#264)
dab2f88 
NethServer/dev#6739
@edospadoni
Copy link
Member

@federicoballarini the PR is merged, you can write test cases then someone will test it.

@edospadoni edospadoni added the testing Packages are available from testing repositories label Mar 29, 2023
@federicoballarini
Copy link
Member Author

Test cases:

  • Create a new address using a specific domain (e.g. test1@domain.local)
  • Save the address and check email is properly created
  • Add a second address: check the specific domain is not selected and the window is coherent (info popup is visible)
  • Select a domain and check the info popup disappear
  • Save and check there are no regressions while creating also wildcard

@nethbot
Copy link
Member

nethbot commented Mar 29, 2023

in 7.9.2009/testing:

@lucagasparini lucagasparini self-assigned this Mar 29, 2023
@lucagasparini
Copy link

All test cases are successfully verified 👍

@lucagasparini lucagasparini added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Mar 29, 2023
@lucagasparini lucagasparini removed their assignment Mar 29, 2023
@nethbot
Copy link
Member

nethbot commented Apr 3, 2023

in 7.9.2009/updates:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug A defect of the software verified All test cases were verified successfully
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@gsanchietti @edospadoni @lucagasparini @federicoballarini @nethbot