[ACTION] Refactor Filter Action
See original GitHub issueWe’ve gotten consistent feedback from users that the Filter actions are confusing to use. I think they’re probably overly complex, and can likely be simplified by flattening the “condition” configurations we currently present and updating some of the names, descriptions, etc.
Here is a non-working prototype of what I have in mind: https://pipedream.com/new?h=tch_wlXfNP
Notes:
- The only reason the
condition
prop is set to have the default it does, is because I want to remove thesecondValue
prop in the case of thesingleValueConditions
(binary conditions), but we don’t support that logic inadditionalProps()
today. - Open to feedback or other suggestions on how to improve the usability
Issue Analytics
- State:
- Created a year ago
- Comments:7
Top Results From Across the Web
Filter Actions - Tableau Help
Typically, a filter action sends information from a selected mark to another sheet showing related information. Behind the scenes, filter actions send data ......
Read more >Chapter 13. The MVC filter pipeline - ASP.NET Core in Action
This chapter covers. The MVC filter pipeline and how it differs from middleware; Creating custom filters to refactor complex action methods; Using authorization ......
Read more >[Solved]-How to refactor simple filter action in Rails?
How to refactor simple filter action in Rails? In rails how to take only the show action to a different path? Rails -...
Read more >Actions · nvim-treesitter/nvim-treesitter-refactor - GitHub
Filter by actor ; Add an option to apply the current scope highlight to the whole line · #49: Pull request #31 synchronize...
Read more >Refactoring Reducers Example | Redux
Structuring Reducers > Refactoring Reducers: Examples of ways to ... return updateObject(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
LGTM, will wait for @vunguyenhung to give it another pass!
@andrewjschuang thanks for making those changes! FYI I made a few slight tweaks and pushed to your branch.
I think the only thing that’s missing is the case sensitive prop – can you add that as an additionalProp for the text conditions, like in the current filter actions?