Pull to refresh

Comments 20

Сучки и задоринки, за которые мы зацепились

Зацепился за сучек неработающей ссылки на демку, в урл закралась "."
Взял сучкарез и отрезал её, пролез)

У меня так и не открывается, если переходить по ссылке, не заходя внутрь статьи.

Видимо сломалось.

Oops. Something went wrong

We can't find the GitHub repository you're looking for.

И полный URL, и только имя_пользователя/репа (кстати, ожидаемый формат никак не описан), а даже просто в URL писал -- все одинаково.

А почему влитые запросы на слияния у вас считаются «открытыми»? Кажется, что это нелогично. Вот что выдаёт поиск «Везде» (Everywhere) в состоянии «открыто» (State = Open):

А переходим в сам запрос на слияние — а он уже влит более недели назад:

Что интересно, если искать среди конкретно запросов на слияние (Pull Requests), то этот запрос находится только среди «закрытых»:

Потому что тут вы не фильтруете по pull requests (search in "Everywhere") и то, что выделено - это комментарий, а не pull request. Текст pull request'а для этого комментария приведён просто для справки. Но раз это интуитивно непонятно, то видимо нужно доработать UI.

Даже если так, у комментария нет никаких статусов открыт / закрыт, следовательно, это статусы объекта, к которому относится комментарий. И мне кажется неправильным показывать комментарии к тем объектам, которые мы явно исключили из поиска фильтром, то есть, комментариев к закрытым запросам на слияние в результатах поиска быть не должно, если мы ищем только среди «открытых».

Либо вообще исключать комментарии из результатов поиска, если стоит какой-то статус (а у комментария статусов нет) и добавить новый фильтр «без статуса» для поиска среди комментариев. Но мне кажется, первый вариант интуитивно понятнее и проще будет.

Мы решили, что нужно распространять фильтр Open / Closed на комментарии тоже. У самих комментариев такого статуса нет, но у родительских объектов он есть, вот его и будем использовать. Делать будем через новую фичу в мантикоре - JOIN. Это будет отличной демонстрацией этой функциональности.

Ну тогда этот фильтр банально работает сейчас неправильно :)

Да. С нашей с вами точки зрения неправильно. С чьей-то (кто считает, что если фильтр неприменим к чему-то, то он должен просто игнорироваться, а усложнять UI не стоит) правильно. В любом случае это open source проект: если кто-то сделает PR до того, как мы сделаем то, что задумали, то мы с радостью его примем. Спасибо, что подсветили эту проблему.

Sign up to leave a comment.

Articles