✨ Added design customization options for welcome emails

[cmraible]

The functionality is already live on main behind the labs flag; this change just promotes the flag to GA so welcome emails customization will be enabled by default.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9edafc80-7abe-481f-96fa-f51172810c11

📥 Commits

Reviewing files that changed from the base of the PR and between 8b4ecc4 and 5cd6061.

⛔ Files ignored due to path filters (1)

• ghost/core/test/e2e-api/admin/__snapshots__/settings.test.js.snap is excluded by !**/*.snap

📒 Files selected for processing (1)

• ghost/core/core/shared/labs.js

🚧 Files skipped from review as they are similar to previous changes (1)

• ghost/core/core/shared/labs.js

---

Walkthrough

The welcomeEmailsDesignCustomization feature flag was moved from the PRIVATE_FEATURES set to the GA_FEATURES set in ghost/core/core/shared/labs.js. As a result, GA_KEYS now includes the flag, WRITABLE_KEYS_ALLOWLIST no longer lists it under private features, and getAllFlags() reports the flag in the GA portion rather than the private portion.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title describes adding design customization for welcome emails, but the actual change is promoting an existing labs flag to GA, not adding new functionality.	Consider revising the title to reflect the actual change, such as 'Promote welcome emails design customization to GA' or 'Make welcome emails customization generally available'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description clearly explains that the functionality is already live and this change promotes the flag to GA, which accurately relates to the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chris-promote-welcome-emails-customization-to-ga

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cmraible]

Reverting to draft, as we've decided to wait on this until Monday

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud