-
Notifications
You must be signed in to change notification settings - Fork 17
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
fix: make the account id optional when getting plans #1
base: main
Are you sure you want to change the base?
Conversation
this means that a user doesn't need to be logged in to view plans
Apologies, i've added a load of testing commits to this! Will fix this :) |
Hi @joshghent - how are you currently packaging your fork? Would be good to get my hands on this while we wait for the merge. Cheers |
@jfloodnet update the import for this repo in before
after
Then just deploy the functions again with |
thanks @joshghent will give a go |
This means that a user doesn't need to be logged in to view plans.
The docs for this endpoint describe the account_id being optional but sending null or not providing this argument causes an error to be thrown. https://usebasejump.com/docs/billing#get-plans