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 cases to email object_arr #8587

Merged
merged 1 commit into from
Aug 10, 2021
Merged

Conversation

tsmgeek
Copy link
Contributor

@tsmgeek tsmgeek commented Mar 3, 2020

Cases parent_type to be pulled for Email Template

@codecov-io
Copy link

codecov-io commented Mar 3, 2020

Codecov Report

Merging #8587 into hotfix-7.10.x will increase coverage by <.01%.
The diff coverage is 0%.

@@                Coverage Diff                @@
##           hotfix-7.10.x    #8587      +/-   ##
=================================================
+ Coverage          10.73%   10.73%   +<.01%     
=================================================
  Files               3229     3229              
  Lines             240855   240851       -4     
=================================================
+ Hits               25858    25860       +2     
+ Misses            214997   214991       -6

@tsmgeek
Copy link
Contributor Author

tsmgeek commented Aug 17, 2020

Actually thinking about it why do we even check against of allowed types, why not just try to load the bean if parent/id are supplied and its in list of beans?

@SuiteBot
Copy link

SuiteBot commented Aug 27, 2020

CLA assistant check
All committers have signed the CLA.

@tsmgeek
Copy link
Contributor Author

tsmgeek commented Jun 16, 2021

Can we merge this in, so simple but would help make things easier in our pipeline.

Dillon-Brown
Dillon-Brown previously approved these changes Jun 17, 2021
Cases parent_type to be pulled for Email Template
Copy link
Contributor

@Dillon-Brown Dillon-Brown left a comment

Choose a reason for hiding this comment

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

Removed the merge commit, aside from that LGTM.

@jack7anderson7 jack7anderson7 added the Status:Assessed PRs that have been tested and confirmed to resolve an issue by a core team member label Jul 18, 2021
@samus-aran samus-aran merged commit efc91fd into salesagility:hotfix-7.10.x Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status:Assessed PRs that have been tested and confirmed to resolve an issue by a core team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants