Repository navigation
Production: название связанной программы в детали проекта - #777
Merged
Merged
Conversation
10 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Что изменено
В production-контракт детали проекта добавлено read-only
partner_program.name. Интерфейс сможет показать конкретную программу в существующем блоке «Основное».Зачем
Проверенный на живом dev API #770 переносится точечно от
master. Dev и production имеют разный выбор программной связи по роли; production scope сохраняется.Реализация
Две строки в
PartnerProgramProjectSerializer: имя берётся из уже выбранного и joinedpartner_program.get_partner_program, permissions, eligible-link queryset и существующий public fallback не меняются. Тесты дополнены проверками имени для manager/expert scope, rename и parity списка/детали.API
GET /projects/{id}/и ответы того же serializer добавляютname; IDs, submission state, поля заявки,can_submit,null, request/validation/errors прежние. Additive backward-compatible contract.UI
В этом PR не менялся; Angular #421 выводит имя и согласованные статусы в desktop/mobile.
Permissions
Без изменений. Сохранены production role-scoped выбор связи и запрет чтения/записи чужих объектов. Новое имя относится к разрешённой пользователю связи.
Проверки
python manage.py test users.tests.test_project_list_program_contract projects.tests.test_program_project_detail_access projects.tests.test_project_programs projects.tests.test_project_crud partner_programs.tests.test_program_project_submit --verbosity 1 --keepdb --noinput— 39 passed.--check, Flake8 на трёх изменённых файлах,git diff --check— passed.f5937b0a0c9d1dcfc5f0950d1cc6d0cc63aee89e.Acceptance criteria
Не проверено
Production runtime после выпуска. Оба полных backend CI прошли; локально выполнены перечисленные targeted/regression tests. Release Prod и последующий production smoke выполняются при выпуске.
Риски / migration
Миграции не нужны. Порядок: merge этого PR → пользователь запускает Release Prod с зафиксированным API ref → frontend merge/autodeploy. Полный dev не переносить. Проверенная база:
f2874390ffd97ba630c1826a0ac179720a039c8a; merge и production deploy выполняет пользователь.