Skip to content

баг-фикс - #42

Merged
Greem4 merged 2 commits into
mainfrom
баг-фикс
Sep 15, 2026

Hidden character warning

The head ref may contain hidden characters: "\u0431\u0430\u0433-\u0444\u0438\u043a\u0441"
Merged

Greem4 merged 2 commits into
mainfrom
баг-фикс

Conversation

@Greem4

@Greem4 Greem4 commented Sep 15, 2026

Copy link
Copy Markdown
Owner

Что меняет

Как проверил

Чек-лист

  • Коммиты по Conventional Commits —
    см. правила проекта
  • ./gradlew :app:testDebugUnitTest зелёный
  • Заметки к релизу лежат в разделе «Не выпущено»
    CHANGELOG.md — их пишет агент по команде
    /changelog, а не автор по памяти в день выпуска
  • Документация обновлена, если поведение изменилось
  • Push-контур, синхронизация YClients и расчёт денег не задеты — либо
    изменение обсуждалось в issue до кода

Связанные issue

Версия

Greem4 and others added 2 commits September 15, 2026 22:09
Система убивает процесс, а задачу оставляет в недавних: тап по уведомлению
воссоздаёт активити со старым savedInstanceState и новым интентом, а
onNewIntent при этом не приходит — экземпляр новый. Разбор extras стоял в
ветке else, поэтому такой тап открывал обычный календарь: ни дня из
напоминания о занятии, ни «О программе» с готовой кнопкой «Обновить».

Повторный разбор того же интента при обычном пересоздании ничего не сдвигает —
applyNotificationExtras поднимает версию deep link только при изменении полей.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
В журнале время занимают не только записями клиентов: слот подписывают одним
именем без карточки клиента («Кац», «Сапожникова») или оставляют себе памятку
(«ДИАГНОСТИКИ, если нет никого…», «если Филиппов отменится — никого не
ставить»). Услуги у такой записи нет, а paid_full YClients ставит в 1 —
оплачивать нечего. Приложение читало это как оплату и приписывало полную
ставку: 10.09.2026 показывало «Занятий 7, итог 10 500 ₽» там, где YClients
начислил 9 000 ₽, а 11.09 — 1 500 ₽ в дне, за который начислено 0.

Признак — услуга: деньги в YClients считаются от неё, и запись без услуги не
приносит там ни рубля. Без услуги paid_full не читается, строка дня получает
префикс NOT_COUNTED_PREFIX, и такая запись не входит ни в «Занятий», ни в
«Проведено», ни в «Заработано», ни в «Ожидается», ни в «Потеряно на отменах».
Статус при этом остаётся настоящим: «не пришёл» так и виден отменой.

Заодно чиню потерю подписей, которые YClients кладёт в комментарий, а не в имя
(«Пирогов», «Сапожникова»): имя выходило пустым, и запись отсеивалась в
collapseDuplicateRecords — в журнале слот занят, в приложении пусто.

Проверено на живых данных: сентябрь — 56 занятий и 84 000 ₽ и в приложении, и
в расчёте YClients.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@Greem4
Greem4 merged commit 05e90db into main Sep 15, 2026
3 checks passed
Greem4 added a commit that referenced this pull request Sep 15, 2026
Шаг «Решить, что выпускаем» ставил KIND=keep, если version.properties попал
в список файлов PR. Две пустые строки в конце файла (PR #42) сошли за подъём
версии: workflow взял 0.2.5 как есть и упал на проверке «тега ещё нет» —
релиз не вышел.

Теперь сравнивается само значение VERSION в main до и после слияния.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Greem4 added a commit that referenced this pull request Sep 15, 2026
Выпуск 0.2.6 руками: автоматический релиз после PR #42 упал на занятом
теге v0.2.5, заметки к релизу с тех пор лежали в «Не выпущено».

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Greem4 added a commit that referenced this pull request Sep 16, 2026
Их добавил PR #42 — из-за них workflow принял файл за поднятую версию.
Сама причина уже вылечена, но мусор в файле оставлять незачем.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant