- Issue created by @phenaproxima
- Merge request !718Create the backend data object for TextFilter โ (Merged) created by phenaproxima
- ๐ฎ๐ณIndia kunal.sachdev
kunal.sachdev โ made their first commit to this issueโs fork.
- ๐บ๐ธUnited States phenaproxima Massachusetts
Fixed merge conflicts and then did the one thing that was causing it to break, which is that the
onFilterChange
handler was not being sent to the TextFilter component. :) - ๐ฎ๐ณIndia narendraR Jaipur, India
Overall, this looks good to me. Just a few suggestions.
- ๐บ๐ธUnited States phenaproxima Massachusetts
I'll ask @tim.plunkett to review/RTBC this -- it seems to be good to go!
- ๐บ๐ธUnited States phenaproxima Massachusetts
Since I mostly didn't write this, tentatively marking it RTBC! The real proof is in the crucible of manual testing that @chrisfromredfin will subject this to.
- First commit to issue fork.
-
chrisfromredfin โ
committed a6200193 on 2.0.x authored by
phenaproxima โ
Issue #3505607: Convert the Search filter to a generic text filter
-
chrisfromredfin โ
committed a6200193 on 2.0.x authored by
phenaproxima โ
- ๐บ๐ธUnited States chrisfromredfin Portland, Maine
cool, this needed a follow-up but that's been filed.
๐ Remove filter definitions from ProjectBrowserSourceBase Active
Automatically closed - issue fixed for 2 weeks with no activity.