Skip to content

Track manual ownership transfers - #1085

Open
cocomarine wants to merge 2 commits into
mainfrom
1951-add-new-column-to-track-manual-transfers
Open

cocomarine wants to merge 2 commits into
mainfrom
1951-add-new-column-to-track-manual-transfers

Conversation

@cocomarine

@cocomarine cocomarine commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Status

What's changed?

  • Added a new column admin_initiated to OwnershipTransfer.
  • Replaced skip_request_email with the new attribute as it does the same thing.

Steps to perform after deploying to production

  • Currently, OwnershipTransfer table in prod is only populated with those from the front-end led transfers.
  • Before deploying, make sure no manual, admin-initiated transfer is added.

@cla-bot cla-bot Bot added the cla-signed label Oct 30, 2026
@github-actions

Copy link
Copy Markdown

Test coverage

94.14% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/41082327814

@macroscopeapp

macroscopeapp Bot commented Oct 30, 2026

Copy link
Copy Markdown

Approvability

Verdict: Would Approve

Macroscope's review found this PR approvable — This records which ownership transfers were initiated administratively while preserving existing email suppression and user-led transfer behavior. The change adds metadata without changing permissions or ownership-transfer rules.

Needs human approval because:

  • approvability is in preview mode for this repository — Macroscope would have approved this PR outside preview mode

You can add or adjust custom eligibility rules. Learn more.

This branch was successfully deployed

1 active deployment
editor-api-p-1951-add-n-wb1ej2 — 15826710 Deployed Oct 30, 2026 by raspberrypiherokubot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants