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

Saurao Solution #24

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

Saurao Solution #24

wants to merge 10 commits into from

Conversation

saurao1999
Copy link

@saurao1999 saurao1999 commented Oct 27, 2023

Post Deployment:-Please Assign the Permission set "NPS Integration Permission" to user.

Limitations:- we can't have a return type for apex class in this solution because of record triggered flow since salesforce doesn't allow Callout operation after DML operation in Same transaction and since as per order of execution in salesforce the salesforce doesn't commit all the changes to org unless it has run all the before and after transaction so the callout is done asynchronously in this solution. Since Salesforce carry asynchronous apex after committing all changes to org and if we try to do the callout after DML operation in synchronous apex we will get the error of "you have uncommitted work pending" so asynchronous solution is suitable with record-triggered flow .
I have created a custom object to track all the error encountered during NPS Callout operation. This will help Admin to know the issue encountered during callout operation.

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