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

fix(move-mail): #MZI-192 fix move mail from parent to a sub folder #104

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

Alex-Delaveau
Copy link
Contributor

@Alex-Delaveau Alex-Delaveau commented Sep 4, 2023

Describe your changes

Fix an issue where subfolder wasnt shown in move mail lightbox if the mail was in the parent folder.

image

Checklist tests

Select a mail from a folder that have subfolder, try to move it and check if the sub folders appear in the lightbox

Issue ticket number and link

MZI-192

Checklist before requesting a review (magic string, indentation, comment/documentation...)

  • I have detailed the tests to do in my feature/fix in order to prevent consequents regressions (must specify in Checklist tests)
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (API Doc etc...) - (must specify in Description for target version)
  • If it is a consequent feature, I have added thorough tests.
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been added to this project (must specify in Description)

@Alex-Delaveau Alex-Delaveau marked this pull request as draft September 4, 2023 15:19
@Alex-Delaveau Alex-Delaveau marked this pull request as ready for review September 5, 2023 08:23
@Alex-Delaveau Alex-Delaveau merged commit a360630 into dev Sep 6, 2023
@Alex-Delaveau Alex-Delaveau deleted the MZI-192 branch September 6, 2023 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants