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

Bugfix/542 emailSendDefinition deploy breaks due to missing email asset #703

Conversation

JoernBerkefeld
Copy link
Contributor

PR details

What changes did you make? (Give an overview)

Is there anything you'd like reviewers to focus on?

previous PR #549 was closed when the branch got renamed. hence this new PR

during deploy tests, after fixing the obvious issues with SenderProfile, DeliveryProfile and SendClassification, I've been getting error 42117. Looking at https://developer.salesforce.com/docs/marketing/marketing-cloud/guide/42000_42999_emailsenddefinition_object.html that error stands for ESD_Invalid ObjectID = Invalid ObjectID

could make sense during an update. but why during create?!

now we get a 42116.... null value.

tested a lot but so far no idea whats wrong now.

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • test scripts updated
  • Wiki updated (if applicable)

Daniel Ventura and others added 17 commits November 9, 2022 17:24
…mailsenddefinition-deploy-breaks-due-to-missing-email-asset
…wont-be-found-on-child-bus' into 542-bug-emailsenddefinition-deploy-breaks-due-to-missing-email-asset
…efinition-deploy-breaks-due-to-missing-email-asset
…senddefinition-deploy-breaks-due-to-missing-email-asset
was already there outside of array
…oap' into 542-bug-emailsenddefinition-deploy-breaks-due-to-missing-email-asset
…oap' into 542-bug-emailsenddefinition-deploy-breaks-due-to-missing-email-asset
…senddefinition-deploy-breaks-due-to-missing-email-asset
…id-during-retrieve-of-triggeredsend' into 542-bug-emailsenddefinition-deploy-breaks-due-to-missing-email-asset
…eve' into 542-bug-emailsenddefinition-deploy-breaks-due-to-missing-email-asset
@JoernBerkefeld JoernBerkefeld added this to the 4.3.2 milestone Feb 3, 2023
@JoernBerkefeld JoernBerkefeld self-assigned this Feb 3, 2023
@JoernBerkefeld JoernBerkefeld changed the base branch from main to develop February 3, 2023 11:02
@JoernBerkefeld JoernBerkefeld marked this pull request as draft February 3, 2023 11:04
@JoernBerkefeld JoernBerkefeld marked this pull request as ready for review February 7, 2023 15:18
@JoernBerkefeld JoernBerkefeld added bug Something isn't working c/emailSend COMPONENT labels Feb 7, 2023
@JoernBerkefeld
Copy link
Contributor Author

latest tests have shown that i can easily duplicate existing emailSendDefinitions now.
whatever blocked us before may have been freak problem that existed on the previous testing BU alone

@JoernBerkefeld JoernBerkefeld changed the title Bugfix/542 emailsenddefinition deploy breaks due to missing email asset Bugfix/542 emailSendDefinition deploy breaks due to missing email asset Feb 7, 2023
@JoernBerkefeld JoernBerkefeld merged commit 93b2f42 into develop Feb 7, 2023
@JoernBerkefeld JoernBerkefeld deleted the bugfix/542-emailsenddefinition-deploy-breaks-due-to-missing-email-asset branch February 7, 2023 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working c/emailSend COMPONENT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] emailSendDefinition deploy breaks due to missing email asset / list
1 participant