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

Changing test to not use Order object #506

Merged

Conversation

daveerickson
Copy link
Contributor

@daveerickson daveerickson commented Feb 18, 2025

I also ran into the issue identified in #505 within my project since my org does not have Orders enabled.

I changed Order object to List Email since that seems to be one of few that do not need to be enabled and has Activities enabled. I am open to refactoring to a different object if List Email is not preferred.


This change is Reviewable

Copy link
Contributor

@daveespo daveespo left a comment

Choose a reason for hiding this comment

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

Thanks -- finding a set of standard object that's available regardless of flavor/generation/type of org is like walking a mine field. I have no affinity to the Order object so we can give ListEmails a try instead.

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @daveerickson)

@daveerickson
Copy link
Contributor Author

Yeah, I was walking describe info for Opportunity to find List Email. :)

Thanks for being flexible, and for the quick response.

@ImJohnMDaniel ImJohnMDaniel merged commit b22fa15 into apex-enterprise-patterns:master Feb 18, 2025
2 checks passed
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