#chris2me_api
feral caveBOT
👋 Welcome to your new thread!
⏲️ We'll be here soon! Typically we respond in a few minutes, but sometimes we might take a bit longer if the server is busy or if you have a particularly tricky question.
⏱️ We close idle threads, which makes them read-only. Once a thread is closed it won't be reopened, but you can always start a new thread if you have another question.
🔗 This thread will always be available, even after it's closed. You can find it again using Discord's search, or you can save this link: https://discord.com/channels/841573134531821608/1507296850846421032
📝 Have more to share? Add more details, code, screenshots, videos, etc. below.
proper trout
karmic knoll
👋 Hi there! Let me take a look
Yeah, I get what you mean, even although I can see the Price was explicitly marked as inactive via the Dashboard — it's still in use on this Subscription.
But I can take this as feedback, and see if this can be made more intuitive
proper trout
Thanks a mill. Because at least on V82 Stripe-go it's not clear the conditions in which this field should be included
Godoc here says include price or price_data but price_data generates a new price. I'm not sure if it still says this for newer versions
karmic knoll
Yeah, that matches the current API docs: https://docs.stripe.com/api/invoices/create_preview#create_invoice-subscription_details-items-price_data
But yeah, given that you're providing the Subscription Item ID already, it would make sense to only provide the quantity
Yeah, I can reproduce that as well. Seems like we could make the documentation more clear there
proper trout
I guess something along the lines of 'price is required when changing price.'
Though I'm still of the opinion the original request we made is a valid one. Seems like the burden is on us to determine whether the field is included based on inspecting the price in question and seeing whether it is active or not.
But that is ok as we've already made the workaround for our particular case
karmic knoll
Yeah, it doesn't really feel intentional. I'm seeing if we can update this behaviour
proper trout
Thanks, I'd like to know whether it'll be addressed at some point.
For us right now on v82 we are sorted with our workaround anyway, thanks.
karmic knoll
I've asked the team to look into not rejecting inactive Prices that are already on the Subscription
If you want to be kept up-to-date on this potential change, we can open a support ticket and update you via email
I've also filed feedback on the docs wording regarding price or price_data being "required"
proper trout
Good stuff, thanks. Yeah if we could open a support ticket that'd be great. Can I DM you my email?
feral caveBOT
Hello @proper trout, we have sent you a direct message, please check it at https://discord.com/channels/@me/1507311800583065630
- 🔗The message has instructions on how to open a direct support case with our Developer Support team, in order to help you more effectively.
karmic knoll
Got it, thanks!