Stripe at some point added restrictions to sending cards openly so instead we need to fetch the payment method using tokens.
Sid was removed as keyword argument from some inputs since it caused errors. Possibly depending on the local stripe version in some manner. Using positional argument instead of keyword works
As a new member when starting a subscription and then delaying the signing of the agreement for a while causes problems with the number of days being added to the membership. This was because the number of days added is stored when the payment is done and not when the subscription actually starts. It caused issues if you signup in one month with some number of days, e.g. february with 28, the signed the agreement later, e.g. may with 31 days. In this example it would add 28 days when it should have added 31. This PR changes the behavior so that the number of days added on the first purchase assumes the worst case for number of days, i.e. that the months added have the maximum number of 31 day months possible. This means that in some cases the membership will be a few days longer than expected, but never shorter.
The tests for subscriptions had problems passing.