search

etianen / django-watson

Full-text multi-table search application for Django. Easy to install and use, with good performance.
BSD 3-Clause "New" or "Revised" License
1.2k stars 129 forks source link

support extra fields to be indexed #278

Open valentijnscholten opened 4 years ago

valentijnscholten commented 4 years ago

By defauilt all CharFields and TextFields are indexed.

You can exclude some on registration by using exclude.

Alternatively you can specify the fields that you want to have indexed, and watson will only index these fields.

I would like the possibility to keep indexing all CharFields and TextFields, but allow for extra fields to be specified on registration. For example sometimes you want to index one extra field or one field from a related model, i.e. 'parent__name'.

Currently you would have to manually list all the fields of model and then add the parent__name to the list and use this combined list at registration. There may be work around to hassle with the django meta internals, but it would be nicer to just be able to provide somthings 'extra_fields=()' on registration.

etianen commented 4 years ago

That sounds useful enough for a MR.

On Sat, 12 Sep 2020 at 12:47, valentijnscholten notifications@github.com wrote:

By defauilt all CharFields and TextFields are indexed.

You can exclude some on registration by using exclude.

Alternatively you can specify the fields that you want to have indexed, and watson will only index these fields.

I would like the possibility to keep indexing all CharFields and TextFields, but allow for extra fields to be specified on registration. For example sometimes you want to index one extra field or one field from a related model, i.e. 'parent__name'.

Currently you would have to manually list all the fields of model and then add the parent__name to the list and use this combined list at registration. There may be work around to hassle with the django meta internals, but it would be nicer to just be able to provide somthings 'extra_fields=()' on registration.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/etianen/django-watson/issues/278, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABEKCETJZSIHU2COE377G3SFNNVZANCNFSM4RJNN2DQ .