Mirror CMP registration contacts and newsletter sync through Relay
还没有人认领这个 Issue。
评估
调研方向
Start at courses/views/registration.py::_save_registration_if_valid and trace how accepted_newsletter is persisted, then read community_base.mail.relay_contacts.RelayContactsClient and configured_contacts_client to see the Relay contacts API the sync should use. Verify the actual website client identity (audience dtc, client dtc-website) against the deployment docs. But this issue is blocked on CMP intake 64569174 landing, demands agent-process artifacts, deploy evidence, and a non-PR direct-push workflow — not something a newcomer can start now.
由索引模型根据 Issue 内容生成。
描述
Intake and outcome
Hub intake 9f153998-97e3-4d7b-9a92-3d82e905b3d0, mirror of CMP intake 64569174. CMP is primary: do CMP first and mirror its accepted registration behavior on this website. Verify how registration synchronizes contacts, consent, and newsletter acceptance (accepted_newsletter), and ensure it goes through the Relay contacts API using the website client and Relay naming.
Authority
- Agent notes
- Development process
- Application boundaries
- Course specification
- Security, privacy, operations
- Selective verification
The explicit intake controls this bounded mirror; these references do not expand its scope.
Research and scope
At website base 35a023da73e9f61173f82e273eea8c001b11830c, courses/views/registration.py::_save_registration_if_valid saves the registration and schedules confirmation mail. The form persists accepted_newsletter; there is no contact synchronization call in that submit function. The shared package already provides community_base.mail.relay_contacts.RelayContactsClient and configured_contacts_client, including POST /api/contacts using Relay configuration.
Trace the actual registration contact/consent/newsletter path. Add the missing synchronization or, if the complete path already complies, record code-path evidence and a Relay log of an approved test upsert and rename only leftovers. Use the established website identity rather than silently substituting the CMP identity. Relay production provisioning documentation names audience dtc, client dtc-website; development deployment documentation still refers to dtc-courses, so establish the actual website scope before claiming live acceptance.
Failures must produce a sanitized log and operator alert and never break registration submit. Existing observability events/logging must not be mistaken for proof of a configured alert. Keep personal data and untrusted provider/exception details out of diagnostics.
Dependencies
Blocked for implementation on CMP intake 64569174: the active primary owner must provide its landed SHA and final contact/consent/newsletter payload and failure behavior. Inspecting a primary draft is allowed; mirroring it before it lands is not. Record the accepted primary receipt and any necessary website-specific identity adaptation.
Sibling website registration sessions run in parallel. Use an isolated fresh-main worktree and rebase on fresh origin/main immediately before each push.
Non-goals
No change to which registration form is canonical, registration form policy, account verification policy, approval behavior, confirmation copy, or unrelated integrations. No deletions, PR, shared git stash, or emails to real users. A live test recipient may only be the single address explicitly allowlisted in the intake; its address is omitted from this public issue.
Acceptance criteria
- Verify website registration contact/consent/newsletter (
accepted_newsletter) synchronization and ensure it uses the Relay contacts API with the website client and Relay naming: no DataMailer-named import, setting, environment variable, or dependency on the registration path. If already compliant, record code-path evidence plus a Relay log of an approved test upsert and only rename leftovers. - Failures are loud through both a sanitized log and operator alert and never break registration submit.
- Tests cover registration to Relay contacts request/payload, its failure path, and a guard against DataMailer-named imports on the registration path. Existing suite remains green.
- Follow repository process, including independent tester and PM acceptance and required verification evidence. Work from an isolated worktree based on fresh
origin/main; rebase on freshorigin/mainimmediately before everyHEAD:mainpush. Push directly with no PR, preserve canonical-form ownership, make no deletions, and remove the task worktree/branch after delivery. - Deploy DEV before prod and cite CI/deploy run IDs and SHAs. Never print secrets. Live email tests use only the intake allowlisted address.
- Produce
/home/alexey/tmp/agent-reports/website-registration-relay-contacts-2026-10-24.mdwith SHAs and the required sanitized Hub-result JSON. Useuvor uv-backed Make targets for Python. Use project-local.tmp/for scratch evidence. - Throwaway containers use
docker run --rmandagent.session=task-hub-4b69d917-add9-4098-962e-bbad05740ad1; install cleanup before creation, clean all created resources including failed starts, and preserve sibling and Coolify resources.
Verification scenarios
Django/integration: submit a valid registration and inspect the contacts request and payload under the website identity, including the accepted primary consent/newsletter mapping. Induce synchronization failure and verify the saved registration/submit remain successful while log and alert are emitted. Exercise the import guard on the actual registration path. Run the existing suite and the repository-required verification plan.
Browser: backend-only changes use the process-required Playwright smoke tier. Preserve existing registration success/error behavior; this issue does not require UI changes. Determine screenshot applicability from the verification plan.
Operations: obtain sanitized Relay-side evidence of the approved test upsert; record the primary and website SHAs and DEV then prod CI/deploy run IDs. Clean only resources created by this task.
- 主要语言
- Python
- 星标
- 0
- 派生
- 0
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
DataTalksClub/website 的其他 Issue
-
bug infra operations P1 testing
难度 2/5 1-3 小时 新手友好度 72/100
DataTalksClub/website#329 ·
-
bug content data-migration documentation events operations P1
难度 2/5 1-3 小时 新手友好度 84/100
DataTalksClub/website#327 ·
-
bug frontend P0 testing
难度 2/5 1-3 小时 新手友好度 90/100
DataTalksClub/website#300 · 7 条评论 ·
-
bug data-migration events P0 testing
难度 2/5 1-3 小时 新手友好度 72/100
DataTalksClub/website#295 · 6 条评论 ·
-
bug infra operations P1
难度 4/5 1-2 天 新手友好度 25/100
DataTalksClub/website#474 · 4 条评论 ·
查看 DataTalksClub/website 的全部 Issue
相似的 Issue
-
area:scripts area:tests bug priority:later
难度 2/5 1-3 小时 新手友好度 72/100
Ikalus1988/MisakaNet#4358 ·
维护者通常 1 天内回复
-
Discovered ports from service discovery are never propagated to modules in Module.load()可能已有人在做 @vishalmore90 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 70/100
OWASP/Nettacker#1828 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 88/100
aws-samples/sample-per-model-bedrock#708 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
-
难度 2/5 1-3 小时 新手友好度 75/100
sjkim1127/Reversecore_MCP#365 ·
维护者通常 1 天内回复