add remote filter to web/vacancies/index

[GeorgeGorbanev]

#33

[davydovanton]

Привет, спасибо за фильтр. Я бы хотел видеть реализацию через search_query параметр. Выглядеть это должно как:

rubyjosbs.dev?query=remote:true
rubyjosbs.dev?query=remote:false

Сможешь убрать remote_query и переделать логику на search_query?

[davydovanton]

Если что - можем созвониться в зуме и сделать эту фичу вместе (считай парное программирование)

[GeorgeGorbanev]

Думаю, я понимаю как это сделать.

1. Положим результат #search_query в инстанс переменную контроллера и добавим expose
2. Внутри operations/list переносим единственное условие на search_query[:remote] вместо параметра
3. Во вью начинаем использовать выставленный из контроллера search_query

Так?

[asusikov]

Правильно я понимаю, что когда мы будем в будущем добавлять новые параметры, то url будем выглядеть примерно так rubyjosbs.dev?query=remote:false&query=location:Нью-Васюки&query=position_type:3?
Посмотрел код и думаю вопрос снимается - url будет такой rubyjosbs.dev?query=remote:false location:Нью-Васюки position_type:3.

[davydovanton]

@GeorgeGorbanev да, выглядит правильно. Стоит проверить все 3 значения true/false/nill. Если что - переименовать/переделать думаю будет просто такое решение.

@asusikov да, будет строчка 1 в 1 как в гитхабе:

[asusikov]

Насчет

Внутри operations/list переносим единственное условие на search_query[:remote] вместо параметра

Лучше будет передавать весь хэш из search_query в контроллере как параметр search_query в Vacancies::Operation::List - он уже есть, но не используется. И добавить пробрасывание этого хэша в query-объект. Что нам это даст - в будущем при добавлении новых параметров мы не будем трогать контроллер, operation и api у query-объекта - только добавим обработку новых параметров в all_with_contact_relation методе у нашего query-объекта

[davydovanton]

@asusikov ага, в этом и была идея как раз, возможно его типизировать (через драй стракт), но это не критично пока

[GeorgeGorbanev]

Сделал все из #38 (comment)
В #39 круто сделан запрос. Его мерджим первее?

[davydovanton]

@GeorgeGorbanev я бы наверно сначала замержил, потом зарефакторил, потом твои изменения замержу