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

Implement inbound E-Mail support #4524

Closed
teosarca opened this issue Aug 28, 2018 · 3 comments
Closed

Implement inbound E-Mail support #4524

teosarca opened this issue Aug 28, 2018 · 3 comments
Assignees
@metasnw
Labels
branch:release bug_please_use_type:bug_instead priority:high topic:CRM type:enhancement

Comments

@teosarca
Copy link
Member

Is this a bug or feature request?

What is the current behavior?

Which are the steps to reproduce?

What is the expected or desired behavior?
@teosarca teosarca self-assigned this Aug 28, 2018
teosarca added a commit that referenced this issue Aug 29, 2018
@teosarca
TimeUtil.asTimestamp ZonedDateTime support
6e560ca 
#4524
teosarca added a commit that referenced this issue Aug 29, 2018
@teosarca
Implement inbound E-Mail support
2a7af15 
#4524
@teosarca teosarca mentioned this issue Aug 29, 2018
Merged
teosarca added a commit that referenced this issue Aug 29, 2018
@teosarca
fixing typo
976d954 
#4524
teosarca added a commit that referenced this issue Aug 29, 2018
@teosarca
fix dependencies
20aa28e 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
minor refactoring
2e9a5f9 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
Util: encode/decode base64 using guava instead of javax.mail
9dc4329 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
use com.sun.mail/javax.mail (same as spring)
85e337f 
instead if javax.mail/mail which is legacy.

This solves the maven enforcer reported issues.

#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
pom.xml: remove the version because it's already managed by spring-bo…
e1a1216 
…ot-dependencies

#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
minor
8e0786e 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
Mailbox: use ClientId and UserId
06cf739 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
drop EMailProcessor (legacy)
09e19d9 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
de.metas.inbound.mail migration scripts
1d92e22 
#4524
teosarca added a commit to metasfresh/metasfresh-dist-legacy that referenced this issue Aug 30, 2018
@teosarca
add de.metas.inbound.mail migration scripts
008a18d 
metasfresh/metasfresh#4524
teosarca added a commit to metasfresh/metasfresh-dist-legacy that referenced this issue Aug 30, 2018
@teosarca
add de.metas.inbound.mail dep. to serverRoot
96f501e 
metasfresh/metasfresh#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
remove mail support from MStore (legacy)
4bcc43f 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
minor
7f4011b 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
organize classes
813de42 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
don't convert the mail content to bytes
5b1128f 
#4524
teosarca added a commit that referenced this issue Aug 30, 2018
@teosarca
WIP/prototype extract mail content from multipart
79cade0 
#4524
@teosarca teosarca mentioned this issue Aug 30, 2018
Closed
teosarca added a commit that referenced this issue Aug 31, 2018
@teosarca
enable remote cache invalidation for C_InboundMailConfig
bbea872 
#4524
teosarca added a commit that referenced this issue Aug 31, 2018
@teosarca
InboundEMailServer: don't reload if config does not really changed
f961cd3 
#4524
teosarca added a commit that referenced this issue Aug 31, 2018
@teosarca
FileUtils.getFileBaseName
fb43d35 
#4524
teosarca added a commit that referenced this issue Aug 31, 2018
@teosarca
fix how we read the mail content; get & save the attachments
a7cc17f 
#4524
teosarca added a commit that referenced this issue Sep 3, 2018
@teosarca
IUserDAO.retrieveUserIdByEMail
9b02602 
#4524
teosarca added a commit that referenced this issue Sep 3, 2018
@teosarca
InboundEMailRepository: set C_Mail.From_User_ID
013cb94 
#4524
@metasnw metasnw self-assigned this Sep 5, 2018
@teosarca teosarca mentioned this issue Sep 7, 2018
Merged
metas-ts added a commit that referenced this issue Sep 10, 2018
@metas-ts
Update ReleaseNotes.md
9de57b3 
[#1040](metasfresh/metasfresh-webui-api-legacy#1040) Some views are failing because #AD_User_ID was not found in context
[#4411](#4411) New Dunning trigger for flatrate terms
[#4561](#4561) QueryStatisticsLogger shall use logger instead of stderr
[#1037](metasfresh/metasfresh-webui-api-legacy#1037) Advanced tooltip support
[#4560](#4560) msv3 related fixes
[#4554](#4554) process to create dunning-PDF with concatenated invoice-PDFs
[#1035](metasfresh/metasfresh-webui-api-legacy#1035) Implement CORS support
[#4516](#4516) New Action "Copy Window"
[#4552](#4552) AD_User is wrong in shipment schedule when using isDropShip
[#1932](metasfresh/metasfresh-webui-frontend-legacy#1932) Mandatory Field not filled Error too early
[#4528](#4528) Can not add new scale price on 5.71
[#4524](#4524) Implement inbound E-Mail support
[#4541](#4541) Return also URL using function de_metas_endcustomer_fresh_reports.Docs_Generics_Org_Report
@metasnw
Copy link
Member

metasnw commented Sep 15, 2018 •
edited
Loading

IT - Scope of first Iteration

  • OK: Setup inbound Email Server
  • OK: c_mail is created for inbound email => window Email
  • OK: email is linked to ad_user
  • OK: request was created
  • OK: reference from request to Email was created
  • NOK: request is not linked to the c_bpartner although the To: ad_user is linked to a partner
  • NOK: I could not find the original e-mail as file
  • NOK: File Attachments are not stored in metasfresh

Issues found beyond scope. Just here for reference

  • Outgoing E-Mails do not create a request
  • Outgoing E-Mails do not store email in the new table c_mail
  • Single E-Mails like from order do use user email settings but to have it harmonized it would be important to use the email outbound settings in order to configure the mailbox on document type level. e.g. send email from sales order via personal e-mail but the ones from invoice window using invoicing@company.com
  • user can not answer to e-mails from request window

teosarca added a commit that referenced this issue Sep 24, 2018
@teosarca
set request's bpartner and user
71c943f 
#4524 (comment)
teosarca added a commit that referenced this issue Sep 24, 2018
@teosarca
fix attachment content type
96545a5 
#4524 (comment)
teosarca added a commit that referenced this issue Sep 24, 2018
@teosarca
set request's bpartner and user
4be0743 
#4524 (comment)
teosarca added a commit that referenced this issue Sep 24, 2018
@teosarca
fix attachment content type
489c182 
#4524 (comment)
teosarca added a commit that referenced this issue Sep 25, 2018
@teosarca
IQuery.listIds(idMapper)
6097cbb 
#4524 (comment)
teosarca added a commit that referenced this issue Sep 25, 2018
@teosarca
BPartnerId.toRepoId
d576c8b 
#4524 (comment)
teosarca added a commit that referenced this issue Sep 25, 2018
@teosarca
cleanup
6b66a0b 
#4524 (comment)
@metasnw
Copy link
Member

metasnw commented Feb 13, 2019 •
edited
Loading

new IT:

IT - Scope of first Iteration

  • OK: Setup inbound Email Server
  • OK: c_mail is created for inbound email => window Email
  • OK: email is linked to ad_user
  • OK: request was created
  • OK: reference from request to Email was created
  • OK: request is not linked to the c_bpartner although the To: ad_user is linked to a partner
  • OK: File Attachments are not stored in metasfresh
  • NOK: Request Date is not set. That one is critical because nobody will find the email ever.

Issues found beyond scope. Just here for reference

  • OUT: I could not find the original e-mail as file. @teosarca we discussed last time to provide the file as msg file so the user can open it in his email client. We can move this to a later time.
  • Outgoing E-Mails do not create a request
  • Outgoing E-Mails do not store email in the new table c_mail
  • Single E-Mails like from order do use user email settings but to have it harmonized it would be important to use the email outbound settings in order to configure the mailbox on document type level. e.g. send email from sales order via personal e-mail but the ones from invoice window using invoicing@company.com
  • user can not answer to e-mails from request window

teosarca added a commit that referenced this issue Feb 13, 2019
@teosarca
set DateTrx when creating the request from inbound email
e9911a9 
#4524
@metasnw
Copy link
Member

metasnw commented Feb 13, 2019 •
edited
Loading

IT on 5.92.1-14034+master

  • OK: Request Date is set. That one is critical because nobody will find the email ever.

@metasnw metasnw closed this as completed Feb 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@metasnw metasnw

Labels
branch:release bug_please_use_type:bug_instead priority:high topic:CRM type:enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@teosarca @metas-ts @metasnw