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

Add the posibility to move mailboxes #3897

Merged
merged 1 commit into from
Nov 9, 2020

Conversation

GretaD
Copy link
Contributor

@GretaD GretaD commented Oct 26, 2020

Signed-off-by: GretaD gretadoci@gmail.com

@ChristophWurst
Copy link
Member

To bring this to life

Then it will properly show for all mailboxes :)

@GretaD GretaD force-pushed the enhacement/move_mailboxes branch 5 times, most recently from 18c14b6 to 9267bc7 Compare November 5, 2020 13:57
@GretaD GretaD force-pushed the enhacement/move_mailboxes branch 2 times, most recently from a26d00d to 01e23af Compare November 9, 2020 15:06
@GretaD
Copy link
Contributor Author

GretaD commented Nov 9, 2020

@ChristophWurst should i also rename the movemodal.vue into movemessagemodal.vue and include here or as a separate PR?

@GretaD GretaD marked this pull request as ready for review November 9, 2020 15:08
@GretaD GretaD requested a review from ChristophWurst November 9, 2020 15:08
@@ -77,6 +77,14 @@ export default {
type: String,
default: t('mail', 'Choose'),
},
pickedMailbox: {
type: Object,
required: false,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ChristophWurst
Copy link
Member

Needs rebase

Signed-off-by: GretaD <gretadoci@gmail.com>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst force-pushed the enhacement/move_mailboxes branch from 01e23af to b49b94c Compare November 9, 2020 19:07
@ChristophWurst
Copy link
Member

Minor detail I noticed while testing #4045

@ChristophWurst ChristophWurst added this to the v1.7 milestone Nov 9, 2020
@ChristophWurst ChristophWurst mentioned this pull request Nov 9, 2020
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants