Поиск по публичным темам и ЛС одновременно

Продолжение обсуждения из Создание/Просмотр и создание разрешений (снова):

Я считаю, что это отличная идея. В настоящее время у нас есть модификатор in:private для поиска личных сообщений, поэтому, возможно, стоит добавить что-то подобное, например in:all или in:everywhere?

9 лайков

@david: Спасибо за создание этой темы (я собирался подождать, пока у меня не будет готов код :slight_smile: ). Вчера и сегодня утром я немного поработал над этим между встречами, и у меня есть рабочий прототип, добавляющий тег поиска in:all: GitHub - adqm/discourse at search_all · GitHub

Несколько моментов, над которыми стоит подумать:

  • Это просто быстрая «хакерская» реализация, и я тестировал её только на небольшом тестовом экземпляре с парой постов.
  • Правильно ли называть тег in:all?
  • Было бы удобно видеть в списке, какие результаты — личные сообщения, а какие — темы.
  • Нужен чекбокс в окне «расширенного поиска».

Я с радостью продолжу работу над этим, когда у меня появится время. Кроме того, я пока совсем не знаком с Ruby, Rails и кодовой базой Discourse, поэтому любая обратная связь и конструктивная критика будут очень кстати!

5 лайков

Отлично! Как только будете готовы, пожалуйста, создайте PR, чтобы другие могли удобнее комментировать.

Думаю, всё в порядке, если у кого-то нет лучших идей? cc @codinghorror?

Это было бы здорово, но пока не стоит об этом беспокоиться. Это можно реализовать в более позднем PR.

Опять же, это было бы удобно, но это не обязательно делать одновременно. В любом случае, эта функция ориентирована в первую очередь на продвинутых пользователей.

6 лайков

После небольшого исследования оказалось, что оно ещё не совсем готово… но я продолжу с этим работать и отправлю PR, как только исправлю известные мне ошибки.

7 лайков

Только что открыл PR. Буду рад мыслям, обратной связи и критике!

7 лайков

in:all подходит, а in:everything — слишком долго печатать, на мой взгляд.

8 лайков

Благодаря @hartz это теперь реализовано :confetti_ball:

Теперь вы можете использовать in:all, чтобы искать свои личные сообщения и публичные темы в одном месте. Например:

11 лайков

Хм. Разве результаты PM не должны иметь конверт PM рядом с результатом? Иначе как узнать, какой из них какой?

12 лайков

Да, это было бы здорово. @hartz уже предлагал это, но я настоял на том, чтобы сначала реализовать базовую функцию.

@hartz, не хочешь попробовать реализовать это во втором PR? Если нет, я могу посмотреть.

8 лайков

Конечно, я с радостью попробую. Мне интересно немного покопаться в кодовой базе (поскольку я пока не совсем уверен, как всё устроено), и это был бы хороший способ сделать это.

Но, скорее всего, пройдут несколько дней, прежде чем я смогу действительно разобраться, так что если ты хочешь взяться за это сейчас — это тоже нормально.

6 лайков

@david: У меня наконец-то появилась возможность посмотреть на это (разделение отображения личных сообщений и тем в результатах поиска). У меня есть локальное, не очень элегантное решение, которое вроде бы работает, но его, вероятно, нужно ещё доработать. Если я считаю, что это пока не готово к полноценному использованию, стоит ли мне открыть отдельную тему в категории разработки или что-то в этом роде, или же лучше сразу создать pull request и использовать его для обсуждения?

4 лайка

Если у вас уже есть рабочий вариант, было бы здорово создать PR — так проще комментировать конкретные части кода. Если хотите, в GitHub есть новая функция «Черновик PR» (нажмите на стрелку рядом с кнопкой «Создать pull request» перед отправкой).

7 лайков

Окей, звучит отлично, спасибо! Я только что отправил PR.

7 лайков

Иконка теперь объединена — спасибо @hartz :slight_smile:

8 лайков

Эта тема была автоматически закрыта через 5 дней. Новые ответы больше не принимаются.