Skip to content

Add flipper group for raspberrypi.org domains - #1084

Merged
jamiebenstead merged 1 commit into
mainfrom
1950-create-flipper-raspberrypi.org-email-group
Oct 30, 2026
Merged

jamiebenstead merged 1 commit into
mainfrom
1950-create-flipper-raspberrypi.org-email-group

Conversation

@jamiebenstead

Copy link
Copy Markdown
Contributor

Adds a flipper group for users with a @raspberrypi.org email address

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

macroscopeapp Bot commented Oct 30, 2026

Copy link
Copy Markdown

Approvability

Verdict: Would Approve

Macroscope's review found this PR approvable — Adds feature targeting for users with verified raspberrypi.org email addresses while retaining school-based targeting. No features are enabled by this patch, and authorization and persistent data remain unchanged.

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.

@github-actions

Copy link
Copy Markdown

Test coverage

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

@raspberrypiherokubot
raspberrypiherokubot temporarily deployed to editor-api-p-1950-creat-7rlhts October 30, 2026 11:48 Inactive

@mwtrew mwtrew left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice one, looks like that wasn't too bad!

@jamiebenstead
jamiebenstead merged commit 5c050d3 into main Oct 30, 2026
10 checks passed
@jamiebenstead
jamiebenstead deleted the 1950-create-flipper-raspberrypi.org-email-group branch October 30, 2026 13:43

This branch was previously deployed

1 inactive deployment
editor-api-p-1950-creat-7rlhts — 84174d75 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.

3 participants