[v1] Search improvements and ack/nack for logged in users

[Patchew-devel] [RFC PATCH 0/6] Search improvements and ack/nack for logged in users

Paolo Bonzini posted 6 patches 5 years, 8 months ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/patchew-ci tags/patchew/20180823200055.2389-1-pbonzini@redhat.com

There is a newer version of this series

api/migrations/0034_auto_20180822_1106.py |  32 +++
api/models.py                             |   7 +
api/rest.py                               |   2 +-
api/search.py                             | 232 ++++++++++++++--------
api/views.py                              |   4 +-
mods/maintainer.py                        |  72 ++++++-
mods/testing.py                           |   2 +-
static/css/base.css                       |   4 +-
www/templates/series-detail.html          |  10 +-
www/views.py                              |   3 +-
10 files changed, 269 insertions(+), 99 deletions(-)
create mode 100644 api/migrations/0034_auto_20180822_1106.py

Expand all Fold all

[Patchew-devel] [RFC PATCH 0/6] Search improvements and ack/nack for logged in users

Posted by Paolo Bonzini 5 years, 8 months ago

Just an RFC that makes things a bit more interesting for logged-in users.
The initial refactoring is not very well tested and not very well covered
by tests too, so...

Paolo Bonzini (6):
  search: split _process_term into separate functions
  search: push handling of +/- to top parsing function, simplify !
  search: add search by result
  search: pass user to search
  www: add back extra_status to series detail
  add Review model

 api/migrations/0034_auto_20180822_1106.py |  32 +++
 api/models.py                             |   7 +
 api/rest.py                               |   2 +-
 api/search.py                             | 232 ++++++++++++++--------
 api/views.py                              |   4 +-
 mods/maintainer.py                        |  72 ++++++-
 mods/testing.py                           |   2 +-
 static/css/base.css                       |   4 +-
 www/templates/series-detail.html          |  10 +-
 www/views.py                              |   3 +-
 10 files changed, 269 insertions(+), 99 deletions(-)
 create mode 100644 api/migrations/0034_auto_20180822_1106.py

-- 
2.17.1

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel

Re: [Patchew-devel] [RFC PATCH 0/6] Search improvements and ack/nack for logged in users

Posted by Fam Zheng 5 years, 8 months ago

On Thu, 08/23 22:00, Paolo Bonzini wrote:
> Just an RFC that makes things a bit more interesting for logged-in users.
> The initial refactoring is not very well tested and not very well covered
> by tests too, so...

Thanks, looks very good! The only thing missing is tests/test_search.py then..

Fam

> 
> Paolo Bonzini (6):
>   search: split _process_term into separate functions
>   search: push handling of +/- to top parsing function, simplify !
>   search: add search by result
>   search: pass user to search
>   www: add back extra_status to series detail
>   add Review model
> 
>  api/migrations/0034_auto_20180822_1106.py |  32 +++
>  api/models.py                             |   7 +
>  api/rest.py                               |   2 +-
>  api/search.py                             | 232 ++++++++++++++--------
>  api/views.py                              |   4 +-
>  mods/maintainer.py                        |  72 ++++++-
>  mods/testing.py                           |   2 +-
>  static/css/base.css                       |   4 +-
>  www/templates/series-detail.html          |  10 +-
>  www/views.py                              |   3 +-
>  10 files changed, 269 insertions(+), 99 deletions(-)
>  create mode 100644 api/migrations/0034_auto_20180822_1106.py
> 
> -- 
> 2.17.1
> 
> _______________________________________________
> Patchew-devel mailing list
> Patchew-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/patchew-devel

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel