svsticky / constipated-koala

Management system for our study association
https://koala.svsticky.nl/
GNU General Public License v3.0
16 stars 4 forks source link

Treat membership payments as a separate entity #1012

Open Riscky opened 2 years ago

Riscky commented 2 years ago

Currently, Koala treats memberships as activities. This had some merits in the past, especially that it showed up in the payments page for people that didn't choose to pay directly. Since this is now made impossible (#988), this is not relevant anymore. Treating the membership payment an activity is weird, and the code handling this is not the clearest to understand. Since there is no reason to keep it this way, we should rewrite this.

Siem2l commented 2 years ago

A possible solution would be creating a stub member and upon completion of the payment converting this stub to an actual member. In case of adding a member manually or not having a transaction fee this payment should be bypassed and instantly creating the member. Date of becoming member could be retrieved from the created_at attribute that every member has and should suffice to give enough information to the board. An alternative is already creating the member and adding a is_member column to every member which becomes true when the transaction has succeeded. Both solutions have their pros and cons so I'm not sure which one would be preferable

Riscky commented 2 years ago

We might also be able to make member creating depend on a successful payment, but I'm not sure if that's the user experience we want to go for.

Siem2l commented 1 year ago

Hmm sounds as a possibility as well but i'd think this would be not entirely legal to allow a transaction before knowing personal information about the buyer

On Wed, 1 Jun 2022 at 15:44, Rick van Hoef @.***> wrote:

We might also be able to make member creating depend on a successful payment, but I'm not sure if that's the user experience we want to go for.

— Reply to this email directly, view it on GitHub https://github.com/svsticky/constipated-koala/issues/1012#issuecomment-1143632703, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC7C7VGASEGJ5EUULV3MSZLVM5SNRANCNFSM5XQV5I4Q . You are receiving this because you commented.Message ID: @.***>