Telegram-Market/telegram-shop
1
1
Fork 0
Code Issues 45 Pull Requests 1 Actions Packages Projects 1 Releases Wiki Activity

fix(admin): dead callback buttons prod_district_/admin_users + no-op placeholders #131

New Issue
Closed
opened 2026-08-05 11:37:41 +00:00 by NW · 0 comments
NW commented 2026-08-05 11:37:41 +00:00
Owner
Copy Link

Баг-репорт: Пересечения/мёртвые callback-кнопки в роутинге

Severity: Medium | Priority: High

Диагноз (кросс-проверка сгенерированных vs зарегистрированных callback)

Найдены 4 сгенерированных callback_data БЕЗ зарегистрированного обработчика + 3 placeholder-кнопки:

  1. prod_district_ — админская Back to Categories (categoryAddHandler) — формат был underscore (ломал multi-word), не зарегистрирован
  2. admin_users — админская Back to User List — не зарегистрирована
  3. current_page / current_quantity / no_action — placeholder-кнопки (давали No handler warns)

Acceptance

  • prod_district_ зарегистрирован (pipe-формат + handleDistrictBack, админ видит disabled локации через getLocationsByCountryAndCityAdmin)
  • admin_users зарегистрирован (handleUserListBack → viewUserPage(0))
  • current_page/current_quantity/no_action → no-op exact routes
  • Аудит: все 78 сгенерированных callback зарегистрированы, пересечений нет
  • Тесты (48 pass)
  • Коммит a5ea99b + push
  • Деплой

Закрыто коммитом a5ea99b (v1.2.6)

## Баг-репорт: Пересечения/мёртвые callback-кнопки в роутинге **Severity:** Medium | **Priority:** High ### Диагноз (кросс-проверка сгенерированных vs зарегистрированных callback) Найдены 4 сгенерированных callback_data БЕЗ зарегистрированного обработчика + 3 placeholder-кнопки: 1. prod_district_ — админская Back to Categories (categoryAddHandler) — формат был underscore (ломал multi-word), не зарегистрирован 2. admin_users — админская Back to User List — не зарегистрирована 3. current_page / current_quantity / no_action — placeholder-кнопки (давали No handler warns) ### Acceptance - [x] prod_district_ зарегистрирован (pipe-формат + handleDistrictBack, админ видит disabled локации через getLocationsByCountryAndCityAdmin) - [x] admin_users зарегистрирован (handleUserListBack → viewUserPage(0)) - [x] current_page/current_quantity/no_action → no-op exact routes - [x] Аудит: все 78 сгенерированных callback зарегистрированы, пересечений нет - [x] Тесты (48 pass) - [x] Коммит a5ea99b + push - [ ] Деплой Закрыто коммитом a5ea99b (v1.2.6)
NW closed this issue 2026-08-05 11:47:05 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
phase::1
Phase 1: Security
 phase::1
Phase 1
 phase::1
Phase 1
 phase::2
Phase 2: Architecture
 phase::2
Phase 2
 phase::3
Phase 3: Admin API
 phase::3
Phase 3
 phase::4
Phase 4: Web UI
 phase::4
Phase 4
 phase::5
Phase 5: Quality
 phase::5
Phase 5
 priority::critical
Critical priority
 priority::critical
Critical priority
 priority::high
High priority
 priority::high
High priority
 priority::medium
Medium priority
 priority::medium
Medium priority
 type::bug
Bug fix
 type::bug
Bug
 type::feature
New feature
 type::feature
Feature
 type::infra
Infrastructure
 type::infra
Infrastructure
 type::refactor
Code refactoring
 type::refactor
Refactoring
 type::security
Security issue
 type::security
Security
 type::testing
Testing
 type::testing
Testing
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
NW
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Telegram-Market/telegram-shop#131
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?