[Refactor] Filter actions
See original GitHub issueWe should make a couple changes to the current Filter action:
- Add an optional “Reason” prop, and set the
default
toCondition met for exiting [or continuing] workflow
- Break out the current Filter action into 3 different actions and get rid of the
continue
prop:
- “Continue workflow based on a condition” (
filter-continue-workflow-condition
) - “End workflow based on a condition” (
filter-end-workflow-condition
) - “End workflow” (
filter-end-workflow
)
Issue Analytics
- State:
- Created a year ago
- Comments:8
Top Results From Across the Web
[ACTION] Refactor Filter Action · Issue #4427 · PipedreamHQ ...
We've gotten consistent feedback from users that the Filter actions are confusing to use. I think they're probably overly complex, and can likely...
Read more >lodash refactor filter function - typescript - Stack Overflow
I have the following code, both components use a lodash filter, how do I abstract the function so I dont repeat the code?...
Read more >Actions defined by AWS Migration Hub Refactor Spaces
Actions Description Access level Resource typ...
CreateEnvironment Grants permission to create an environment Write
DeleteEnvironment Grants permission to delete an environment Write environment*
DeleteEnvironment Grants permission...
Read more >The Best Refactoring You've Never Heard Of - YouTube
Letting a search procedure take an arbitrary filter function, instead of ... a user stop in the middle of an action — and...
Read more >Refactoring Reducers Example | Redux
switch (action.type) { case 'SET_VISIBILITY_FILTER': { return Object.assign({}, state, { visibilityFilter: action.filter } ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Done
@dannyroosevelt can you approve the fix I pushed yesterday to the PR?