Skip to content

fix: platform plan name is always uppercase#15215

Merged
ThyMinimalDev merged 1 commit intomainfrom
fix-platform-billing-plan-name
May 28, 2024
Merged

fix: platform plan name is always uppercase#15215
ThyMinimalDev merged 1 commit intomainfrom
fix-platform-billing-plan-name

Conversation

@ThyMinimalDev
Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes an issue where we set the platform plan to "none" because if wasn't passed in uppercase and matching the value in constants

@graphite-app graphite-app Bot requested a review from a team May 28, 2024 08:01
@github-actions
Copy link
Copy Markdown
Contributor

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

@keithwillcode keithwillcode added core area: core, team members only platform Anything related to our platform plan labels May 28, 2024
@dosubot dosubot Bot added the 🐛 bug Something isn't working label May 28, 2024
@graphite-app graphite-app Bot requested a review from a team May 28, 2024 08:02
@vercel
Copy link
Copy Markdown

vercel Bot commented May 28, 2024

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

Name Status Preview Comments Updated (UTC)
ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 28, 2024 8:03am

@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented May 28, 2024

Graphite Automations

"Add foundation team as reviewer" took an action on this PR • (05/28/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 • (05/28/24)

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

@ThyMinimalDev ThyMinimalDev added this pull request to the merge queue May 28, 2024
Merged via the queue into main with commit 17c4a10 May 28, 2024
@ThyMinimalDev ThyMinimalDev deleted the fix-platform-billing-plan-name branch May 28, 2024 08:35
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

🐛 bug Something isn't working 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