Дайджест разработки #3, 13.04.20 — 19.04.20

1. По просьбе vbltyt починена сортировка произведений в разделе «Входит в...» на странице произведений. До этого порядок был около-случайный, теперь сортируются по дате от более ранних к более новым. Пример

2. По просьбе БорЧ-а исправлена сортировка изданий. Суть проблемы: у некоторых изданий дата публикации известна с точностью до месяца, но в списке изданий цикла, например, они сортировались без учета месяца, только по году. Исправлено, теперь месяц тоже учитывается (во всех списках изданий на странице произведения). Пример

3. Многие пользователи (и lomakos, который не поленился отписаться) обратили внимание, что сломано отображение списка похожих авторов в библиографии автора — список пересекался с самой библиографией. Исправлено.

4. Некоторый прогресс в Go-API:

5. 1. Доделан аплоад аттачей к сообщениям и черновикам в форуме

6. 2. Исправлена пагинация списка статей в блоге (как-то я умудрился ее пропустить в свое время, выдавался весь список)

7. 3. В списке статей больше не выводится текст статьи, чтобы клиенты не грузили лишнее

8. 4. Список книжных полок пользователя перенесен из Perl API

9. 5. Так же, как и создание первичных полок. Если кто помнит, после регистрации у пользователя есть возможность нажатием кнопки создать 5 полок: «Мои книги», «Продаю», «Куплю», «Прочитать», «Ожидаю». Теперь это работает и в Go-API (пока в тестовом режиме).

10. ilyakokhan в Android-клиенте добавил обработку тегов с выравниванием (пригодится, например, в выводе изображений в статьях) и черновики сообщений в форуме.

11. creator поправил отображение блока «В продаже» у изданий в разделах «Новинки фантастики» и «Планы издательств»

PS Немного деталей внутренней работы с кодом. Все задачи (новый функционал, исправление багов и пр.) проходит через обязательную стадию ревью: создается так называемый Pull request с набором изменений в коде, который должен посмотреть кто-то из других программистов. Если возражений нет, код сливается в основную ветку и заливается на сервер (после этого вы и видите изменения). Если нет — вносятся изменения, пока ревьюер не будет удовлетворен качеством кода. В случае Go-API у нас с nseven92 никаких проблем нет, все изменения ревьюятся в пределах одного дня. К сожалению, для сайта это не работает, поэтому было решено внедрить простое правило: если Pull request висит без ревью более 2 суток, он сливается в основную ветку без уведомления. Именно поэтому, даже если какой-то баг исправлен прямо в день, когда о нем сообщили (а чаще всего так и есть), на сайте этот фикс появится не раньше, чем через пару дней. Скорее всего. Если не случится чуда и не очнутся от вековечного сна Ктулху, Йог-Сотот и Азатот creator, vad и Demiurg.

PPS creator-у закон не писан, он обходится без ревью и это очень печально.