Skip to content

Conversation

@christian-byrne
Copy link
Contributor

@christian-byrne christian-byrne commented Oct 19, 2025

Summary

Backport of PR #6144 to the rh-test branch.

This adds build time feature flags system starting with a flag that indicates whether subscription is required to use the app. This is only used on cloud.

Changes

  • Added build feature flags system via __BUILD_FLAGS__ global
  • Added REQUIRE_SUBSCRIPTION flag that can be set via environment variable
  • Conditionally load subscription-related components based on the flag
  • Updated run button logic to respect the subscription requirement flag
  • Updated settings UI to show subscription panel only when required

┆Issue is synchronized with this Notion page by Unito

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Oct 19, 2025
@github-actions
Copy link

github-actions bot commented Oct 19, 2025

🎭 Playwright Test Results

🕵🏻 No test results found

⏰ Completed at: 10/19/2025, 07:09:00 PM UTC

📊 Test Reports by Browser

  • chromium: Deployment failed
  • chromium-2x: Deployment failed
  • chromium-0.5x: Deployment failed
  • mobile-chrome: Deployment failed

🎉 Click on the links above to view detailed test results for each browser configuration.

## Summary

Adds build time feature flags system starting with a flag that indicates
whether subscription is required to use the app. This is only used on
cloud.

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6144-make-require-subscription-toggleable-in-build-2916d73d3650813bb140c5e96bcce1ce)
by [Unito](https://www.unito.io)
@christian-byrne christian-byrne force-pushed the backport-6144-to-rh-test branch from d6ae84a to 75b757c Compare October 19, 2025 19:03
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Oct 19, 2025
@christian-byrne christian-byrne changed the title Backport: make 'require subscription' toggleable in build to rh-test [backport rh-test] make 'require subscription' toggleable Oct 19, 2025
@christian-byrne christian-byrne added the backport Backporting a PR onto a release candidate label Oct 19, 2025
@christian-byrne christian-byrne merged commit b210e63 into rh-test Oct 19, 2025
14 of 17 checks passed
@christian-byrne christian-byrne deleted the backport-6144-to-rh-test branch October 19, 2025 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:cloud backport Backporting a PR onto a release candidate size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant