Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: optionally require plans for provisioning #1087

Merged
merged 2 commits into from
Nov 7, 2023

Conversation

travis
Copy link
Member

@travis travis commented Nov 7, 2023

We need this to ship billing, to stop users without a payment provider plan from provisioning new spaces.

I've added a flag to the ProviderServiceContext that lets us disable this requirement - this is extremely useful in testing and will be helpful as we roll this feature out - ie, w3infra can disable this requirement until we are ready to launch.

We need this to ship billing, to stop users without a payment provider plan from provisioning new spaces.

I've added a flag to the `ProviderServiceContext` that lets us disable this requirement - this is extremely useful in testing and will be helpful as we roll this feature out - ie, w3infra can disable this requirement until we are ready to launch.
Copy link
Member

@alanshaw alanshaw left a comment

Choose a reason for hiding this comment

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

Personally, I'd name this var something more like allowMissingPaymentPlan.

@travis
Copy link
Member Author

travis commented Nov 7, 2023

ah I like it - I'd like false and undefined here to be equivalent so I'm going to go with requirePaymentPlan

@travis travis merged commit b24731b into main Nov 7, 2023
3 checks passed
@travis travis deleted the feat/no-plan-no-provision branch November 7, 2023 22:00
travis pushed a commit that referenced this pull request Nov 7, 2023
🤖 I have created a release *beep* *boop*
---


##
[7.2.0](upload-api-v7.1.2...upload-api-v7.2.0)
(2023-11-07)


### Features

* add usage/report capability
([#1079](#1079))
([6418b4b](6418b4b))
* optionally require plans for provisioning
([#1087](#1087))
([b24731b](b24731b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants