Daniel Axtens <d...@axtens.net> writes: > Stephen Finucane <step...@that.guru> writes: > >> On Wed, 2018-04-11 at 17:13 +0100, Stephen Finucane wrote: >>> When writing git-pw, it was assumed that one could specify a filter >>> multiple times and it would be applied that way. This turned out to be a >>> false assumption and only the latest filter is applied. This is >>> confusing and prevents a number of possible usecases. Fix this. >> >> Will I respin this or are you happy to address the comments at merge >> time, Daniel? > I can do it at merge this morning. I'll let you know if anything goes > wrong and I need a respin.
Tweaked, merged, pushed; let me know if I've stuffed anything up dramatically. Are we right to do an rc2 now that we have every feature we want? Regards, Daniel > > Regards, > Daniel > >> >> Stephen >> >>> Stephen Finucane (7): >>> REST: Use UserFilter for patch delegates >>> REST: Simplify ModelMultiChoiceField >>> REST: Use ModelMultipleChoiceField >>> REST: Remove ProjectFilterMixin >>> REST: Rename Filter -> FilterSet >>> REST: Use ModelMultipleChoiceField for other fields >>> REST: Use DRF-specific filterset >>> >>> patchwork/api/bundle.py | 4 +- >>> patchwork/api/check.py | 4 +- >>> patchwork/api/cover.py | 4 +- >>> patchwork/api/event.py | 4 +- >>> patchwork/api/filters.py | 162 >>> ++++++++++++--------- >>> patchwork/api/patch.py | 4 +- >>> patchwork/api/series.py | 4 +- >>> patchwork/tests/api/test_patch.py | 15 +- >>> .../improved-rest-filtering-bf68399270a9b245.yaml | 10 +- >>> 9 files changed, 129 insertions(+), 82 deletions(-) >>> _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork