Skip to content

CAMEL-14588: New setting to allow mails to be moved after processing#3587

Merged
davsclaus merged 1 commit intoapache:masterfrom
mash-sap:masterForMoveTo
Feb 19, 2020
Merged

CAMEL-14588: New setting to allow mails to be moved after processing#3587
davsclaus merged 1 commit intoapache:masterfrom
mash-sap:masterForMoveTo

Conversation

@mash-sap
Copy link
Contributor

New setting to allow mails to be moved after processing

Copy link
Contributor

@oscerd oscerd left a comment

Choose a reason for hiding this comment

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

Please add some documentation about this possibility in src/main/docs adoc file

@oscerd
Copy link
Contributor

oscerd commented Feb 18, 2020

And please run the build on camel-mail with the profile sourcecheck enabled and fix the warnings you'll get the github action reports 4 errors.

@davsclaus
Copy link
Contributor

The moveTo option must also be add as configuration option, find the copyTo option and do similar code.

@mash-sap
Copy link
Contributor Author

The moveTo option must also be add as configuration option, find the copyTo option and do similar code.

I added the moveTo option to MailConfiguration.java similar to the copyTo commit (b5be70c). Is there any other place where it has to be added?

@mash-sap
Copy link
Contributor Author

Please add some documentation about this possibility in src/main/docs adoc file

Done

@mash-sap mash-sap requested a review from oscerd February 19, 2020 17:04
@davsclaus davsclaus merged commit 41cb193 into apache:master Feb 19, 2020
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