Skip to content

[PB-6055]: add referral-access feature flag with limit guard#999

Merged
terrerox merged 3 commits into
masterfrom
feature/cello-flag
Mar 23, 2026
Merged

[PB-6055]: add referral-access feature flag with limit guard#999
terrerox merged 3 commits into
masterfrom
feature/cello-flag

Conversation

@terrerox
Copy link
Copy Markdown
Contributor

@terrerox terrerox commented Mar 20, 2026

Gate referral endpoints behind a boolean feature flag using the existing feature-limit system. Adds GET /referral/enabled to check eligibility and protects POST /referral/token with the FeatureLimit guard. Includes migration to seed the limit disabled by default for all tiers with a user override for demo.user@cello.so.

@terrerox
feat(referral): add referral-access feature flag with limit guard
84831a3 
  Gate referral endpoints behind a boolean feature flag using the existing feature-limit system. Adds GET /referral/enabled to check eligibility
  and protects POST /referral/token with the FeatureLimit guard. Includes migration to seed the limit disabled by default for all tiers with a
  user override for demo.user@cello.so.
@terrerox terrerox self-assigned this Mar 20, 2026
@jzunigax2 jzunigax2 added the migration-required A migration is included in the PR label Mar 20, 2026
@terrerox terrerox requested a review from sg-gs March 20, 2026 19:02
jzunigax2
jzunigax2 previously approved these changes Mar 20, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@jzunigax2 jzunigax2 left a comment

Choose a reason for hiding this comment

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

looks good, wait on @sg-gs to run the migration

apsantiso
apsantiso reviewed Mar 20, 2026
View reviewed changes
Comment thread migrations/20260319120000-add-referral-access-limit.js Outdated
@terrerox terrerox requested review from apsantiso and jzunigax2 March 20, 2026 19:14
@jzunigax2 jzunigax2 removed the migration-required A migration is included in the PR label Mar 20, 2026
@terrerox terrerox force-pushed the feature/cello-flag branch from 64b4658 to 415e2a1 Compare March 20, 2026 19:25
@terrerox terrerox added the migration-required A migration is included in the PR label Mar 20, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 20, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

sg-gs
sg-gs approved these changes Mar 23, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@sg-gs sg-gs left a comment

Choose a reason for hiding this comment

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

Running migration @terrerox

@sg-gs
Copy link
Copy Markdown
Member

sg-gs commented Mar 23, 2026

Migration run @terrerox

Sequelize CLI [Node: 22.16.0, CLI: 6.6.3, ORM: 6.37.7]

Loaded configuration file "src/config/sequelize.js".
Using environment "production".
== 20260319120000-add-referral-access-limit: migrating =======
== 20260319120000-add-referral-access-limit: migrated (0.076s)

Done in 1.46s.

@terrerox terrerox merged commit 089203c into master Mar 23, 2026
13 checks passed
@terrerox terrerox deleted the feature/cello-flag branch March 23, 2026 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sg-gs sg-gs sg-gs approved these changes

@jzunigax2 jzunigax2 jzunigax2 approved these changes

@apsantiso apsantiso apsantiso approved these changes

Assignees

@terrerox terrerox

Labels

migration-required A migration is included in the PR ready-for-preview

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@terrerox @sg-gs @jzunigax2 @apsantiso