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

Invocable approval helper #1252

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Henk3000
Copy link
Contributor

@Henk3000 Henk3000 commented Nov 1, 2022

I wanted to use the flow action as shown on this article:
https://unofficialsf.com/use-flow-to-approve-reject-and-cancel-approval-processes/

As I now see in the comments the suggested code does not work, I investigated further and saw room for improvement.
I created 2 classes to be able to use the invocable approval actions
I removed the original flow example as it wasn't even calling the invocable action that was created in the article.

Other than that, it still works as described in the article

@alexed1
Copy link
Owner

alexed1 commented Dec 14, 2022

Hi, Henk. This looks like a great improvement. What do you think of the idea of keeping the same name 'ResolveApprovalRequest'? We generally prefer to upgrade actions in place. It keeps things clearer.

@alexed1
Copy link
Owner

alexed1 commented Dec 14, 2022

I realize now that Tamar also provide a pull request with fixes, at #1281.
I went ahead and merged her PR, but I'd still like to include your improvements if we can do it without changing the name of the action.

@alexed1
Copy link
Owner

alexed1 commented Apr 10, 2023

Hi, checking back again. Let me know if you'd like to discuss this PR.

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.

2 participants