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

Fixed #8177 & related bugs to show the correct "From" email information #8254

Closed
wants to merge 1 commit into from

Conversation

thingumajig42
Copy link

Also fixes:
-#7407-"Users may send as themselves" broken - Invalid address: (punyEncode)
-#7692-From adress incorrect when replying to email

Reintroduced the code that was removed as a fix for #7407 and fixed it
by checking for the case within the condition.

Resolves: #8177, #7692, #7407

Description

Reintroduced the code that was removed which was causing issues #8177 & #7692 as a fix for #7407 and fixed the #7407 issue by checking for the case within the condition.

Motivation and Context

Fixed bugs.

How To Test This

Please look at the resolved issue for more information on testing this fix.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Final checklist

  • My code follows the code style of this project found here.
  • My change requires a change to the documentation.
  • I have read the How to Contribute guidelines.

…il information

Also fixes:
 -salesagility#8177-Email sent from case does not have the correct "From"
information
 -salesagility#7407-"Users may send as themselves" broken - Invalid address:
(punyEncode)
 -salesagility#7692-From adress incorrect when replying to email

Reintroduced the code that was removed as a fix for salesagility#7407 and fixed it
by checking for the case within the condition.

Resolves: salesagility#8177, salesagility#7692, salesagility#7407
@thingumajig42
Copy link
Author

Since the travis check has been pending for 2+ days, the recommendation was to open a new PR according to https://suitecrm.com/suitecrm/forum/developer-help/28099-pull-request-waiting-on-travis-status-for-1-day-https-github-com-salesagility-suitecrm-pull-8254
So, closing this PR and opening a new one.

@thingumajig42 thingumajig42 deleted the bugfix_8177 branch November 16, 2019 09:43
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.

None yet

1 participant