Skip to content

feat: enable PBAC feature flag globally via migration#27376

Merged
sean-brydon merged 1 commit intomainfrom
devin/1769677733-enable-pbac-globally
Jan 29, 2026
Merged

feat: enable PBAC feature flag globally via migration#27376
sean-brydon merged 1 commit intomainfrom
devin/1769677733-enable-pbac-globally

Conversation

@sean-brydon
Copy link
Copy Markdown
Member

What does this PR do?

Enables the PBAC (Permission-Based Access Control) feature flag globally via a database migration. This updates the Feature table to set enabled = true for the 'pbac' slug.

The PBAC feature was previously added with enabled: false in migration 20250521070158_add_pbac_feature_flags. This migration enables it for all instances.

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A - no documentation changes needed.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  1. Run the migration: yarn workspace @calcom/prisma db-migrate
  2. Verify the feature flag is enabled:
    SELECT * FROM "Feature" WHERE slug = 'pbac';
    Should show enabled = true
  3. Monitor e2e tests for any PBAC-related failures after enabling

Human Review Checklist

  • Confirm enabling PBAC globally is the intended behavior
  • Verify all PBAC-related code paths work correctly with the feature enabled
  • Check if there are any known edge cases or issues when PBAC is enabled globally
  • Ensure monitoring is in place for post-deployment

Checklist

  • I have read the contributing guide
  • My code follows the style guidelines of this project
  • I have checked if my changes generate no new warnings
  • My PR is appropriately sized (4 lines changed)

Link to Devin run: https://app.devin.ai/sessions/efe54642f0d44c5e9b1413c9fdee7485
Requested by: @sean-brydon

@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions github-actions bot added the ❗️ migrations contains migration files label Jan 29, 2026
@sean-brydon sean-brydon marked this pull request as ready for review January 29, 2026 09:33
@sean-brydon sean-brydon requested a review from a team as a code owner January 29, 2026 09:33
@graphite-app graphite-app bot requested a review from a team January 29, 2026 09:33
@graphite-app graphite-app bot added consumer core area: core, team members only labels Jan 29, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@sean-brydon sean-brydon merged commit 1f8f134 into main Jan 29, 2026
89 of 90 checks passed
@sean-brydon sean-brydon deleted the devin/1769677733-enable-pbac-globally branch January 29, 2026 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consumer core area: core, team members only ❗️ migrations contains migration files ready-for-e2e size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants