ari-pep / PepCoopMembership

a webapp/form for people to join Cultural Commons Collecting Society (C3S) SCE
GNU General Public License v3.0
0 stars 1 forks source link

Button to Export Members and applyies as CSV #76

Closed buffstop closed 6 years ago

buffstop commented 6 years ago

Implement a button in backend (or introduce a certain URL is fine also to not mess with the UI), that offers an CSV export of all member and applyees for download. The following fields should be exported as columns, ideally with column header titles, in given order:

id membership_number firstname lastname address1 address2 postcode city country email num_shares member_type fee entry_fee payment_method payment_sdd_iban payment_sdd_bic payment_sdd_bankname date_of_submission email_is_confirmed signature_received payment_received accountant_comment date_of_birth membership_loss_type

Please have a look with your eagles eyes, most probably I forgot something obviously.

ari-pep commented 6 years ago

what's NULL sopposed to be? (i.e. membership no is initially NULL)

did you leave these out on purpose?

otherwise, it looks fine to me

buffstop commented 6 years ago

what's NULL sopposed to be? (i.e. membership no is initially NULL)

I am not sure what's the best value here for sheets. Please simply use empty string, 1.1.1970, other common default values. I might ask to change that later though :-/

did you leave these out on purpose?

My idea is to:

Thus imo the following is not of interest. Only that we received payment/signature.

So yes, I intentionally excluded the following: signature_received_date signatue_confirmed_date payment_received_date payment_confirmed_date certificate_email,+date

I was not aware of this: membership_loss_date,type

What is the membership_loss_type? That sounds interesting.

ari-pep commented 6 years ago

regarding datasheet explosion: It might be helpful to separate (two csvs):

ack wrt null, excluding: 1.1.1970 is particularly evil. To unix-people it stands out of course, but to everybody else it seems like a normal new year. We have people with birthday in 1970 already and I needed to double-check wether that's not some error

membership_loss_type: one of [resignation, expulsion, death, bankruptcy, winding-up, shares_transfer] where the last is the transfer of all remaining shares. Some apply only to humans (death), some only to legal persons (winding-up). Check out editing an existing member.

buffstop commented 6 years ago

regarding datasheet explosion: It might be helpful to separate (two csvs):

maybe YAGNI. Lets implement only what we currently need.

... excluding: 1.1.1970 is particularly evil...

Aggree. Use "0000-00-00", adapted to the date format of your choice.

membership_loss_type: one of [resignation, expulsion, death, bankruptcy, winding-up, shares_transfer]

Please export this also. Updated description.

ari-pep commented 6 years ago

JFYI: 1970-01-01 is currently in production so this is going to be 1970-01-01 -> 0000-00-00

otherwise ack and afaics ready to go

ari-pep commented 6 years ago

@buffstop: there's now /admin/admin.csv on testing, please check wether it matches your intentions

buffstop commented 6 years ago

testing is down.

Service Unavailable

The server is temporarily unable to service your request due to maintenance downtime or capacity problems. Please try again later.

ari-pep commented 6 years ago

my fault, sorry - up again

buffstop commented 6 years ago

Looks good to me. Only thing missing is "id". Also I changed my mind with certain fields and removed them from the list. Sorry. Left ToDo:

payment_received_date email_confirm_mail_date membership_loss_date membership_date

ari-pep commented 6 years ago

changes are in testing

buffstop commented 6 years ago

Looks fine for a first version. It will take me some time to deal with it and find potential problems. I will create new tickets in case of problems. Thx!

buffstop commented 6 years ago

PS: Please tripple-check that the export is not available without certificate or minor tricking.

buffstop commented 6 years ago

Export is (currently?) unavailable on production (join). 404 Not Found

ari-pep commented 6 years ago

weird - the code was not in production, but I'm pretty sure at some point it was. Anyway: fixed