Skip to content

fix: сайт мероприятия и контакт в запросе внешней регистрации - #801

Merged
TThanos3000 merged 2 commits into
devfrom
fix/dev-external-registration-request-20261020
Oct 20, 2026
Merged

TThanos3000 merged 2 commits into
devfrom
fix/dev-external-registration-request-20261020

Conversation

@TThanos3000

@TThanos3000 TThanos3000 commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Что изменено

Запрос внешней регистрации принимает сайт мероприятия и необязательные комментарий/контакт. Администратор видит сайт и контакт в Django, а фактическая ссылка регистрации после подключения остаётся отдельным external_url.

API и совместимость

POST programs/{id}/management/registration/connection/ сохраняет прежний url, делает comment необязательным и добавляет необязательный contact (500 символов). Settings response добавляет connection_website_url и connection_contact. Старый клиент продолжает отправлять прежний payload. Неизвестные поля и некорректные значения отклоняются до записи. Права организатора/администратора, блокировка программы и запрет самостоятельного подключения не изменяются. Уже подключённая форма запросом не перезаписывается.

UI

Angular PR PROCOLLAB-github/procollab#464 обновляет подписи полей и текст обращения/обратной связи. Backend выкладывается первым. Обращения остаются в очереди Django admin; автоматические письма этим изменением не добавляются.

Проверки

  • PostgreSQL: 31 целевой тест прошёл, включая необязательные детали, сохранение, validation/no partial writes, object permissions, защиту подключённой формы и сохранение старых URL при миграции.
  • Flake8 и makemigrations --check прошли; миграция0025 успешно применена локально.
  • UI browser verification прошла на1440/390/360 с изолированным API (см. связанный frontend PR); actual DEV migration/runtime smoke — после deployment.

Migration / rollout

0025 добавляет два необязательных поля. Сайт старых ожидающих запросов копируется из прежнего URL; опубликованные ссылки и пользовательские данные не переписываются. При откате кода схему0025 оставляем, чтобы сохранить данные новых обращений. Сначала DEV API/migration/runtime/Celery, затем Angular. Production исключён.

DEV — проверено после выкладки

API PR801 и Angular PR464 выложены обычным merge в dev. Миграция0025/runtime-права/HTTP smoke/Celery прошли; frontend format/lint/2280tests/build/rsync прошли. На опубликованных assets проверены1440/390/360px: все3 сценария прошли, включая отправку без комментария/контакта и сохранение отдельных деталей; ошибок runtime/5xx нет. Ещё6 анонимных GET-проверок реального dev прошли. Записи и почта в QA изолированы; production не изменён. Desktop/mobile макеты синхронизированы в Figma.

@TThanos3000
TThanos3000 marked this pull request as ready for review October 20, 2026 08:54
@TThanos3000
TThanos3000 merged commit f453296 into dev Oct 20, 2026
3 checks passed
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.

2 participants