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

Initial Commit #22

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

ozkanyanik
Copy link

There are two options that can be considered for the business layer to call from the Invocable Method:

  • Call the NPS Service with the future annotation.
  • Call the NPS Service from a batchable apex class.

I added both classes with test classes.

Limitations-Possible problems:

  • The running user must have the corresponding Invocable Apex class security set in their user profile or permission set.
  • Callout limits per transaction must be considered.(Which is maximum of 100 in this case)
  • The default timeout is 10 seconds. Increasing the timeout can be considered.
  • NPS Api allows sending up to 30 Orders at once. This limit definitely should considered.
  • A custom checkbox field can be added to the Order object to identify synced orders.
  • Email Apex Action can be invoked on the flow to inform admin users.

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.

1 participant