Skip to content

refactor: only check platform billing for platforms#15507

Merged
supalarry merged 2 commits intomainfrom
dont-check-non-platform-team
Jun 20, 2024
Merged

refactor: only check platform billing for platforms#15507
supalarry merged 2 commits intomainfrom
dont-check-non-platform-team

Conversation

@supalarry
Copy link
Copy Markdown
Contributor

Problem

If user belongs to a non platform org we are sending requests to v2/billing/:teamId/check which results in 403:
image

Solution

Disable useCheckTeamBilling query calling v2/billing/:teamId/check if user's organization is not platform.

@graphite-app graphite-app Bot requested a review from a team June 20, 2024 10:18
@dosubot dosubot Bot added the platform Anything related to our platform plan label Jun 20, 2024
@keithwillcode keithwillcode added the core area: core, team members only label Jun 20, 2024
@graphite-app graphite-app Bot requested a review from a team June 20, 2024 10:19
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Jun 20, 2024

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (06/20/24)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add platform team as reviewer" took an action on this PR • (06/20/24)

1 reviewer was added to this PR based on Keith Williams's automation.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
ai ⬜️ Ignored (Inspect) Visit Preview Jun 20, 2024 10:22am
cal ⬜️ Ignored (Inspect) Visit Preview Jun 20, 2024 10:22am
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Jun 20, 2024 10:22am

@supalarry supalarry enabled auto-merge June 20, 2024 10:27
@supalarry supalarry added this pull request to the merge queue Jun 20, 2024
Merged via the queue into main with commit f760a8a Jun 20, 2024
@supalarry supalarry deleted the dont-check-non-platform-team branch June 20, 2024 10:42
p6l-richard pushed a commit to p6l-richard/cal.com-fork that referenced this pull request Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only platform Anything related to our platform plan

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants