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

Backend: error shares/detail #44

Open buffstop opened 6 years ago

buffstop commented 6 years ago

PRIO: not before opening gig

Repro: In backend:

Expected: Some details in regards of the shares are shown.

Actual: Error is shown: Internal Server Error

The server encountered an unexpected internal server error

(generated by waitress)

###

self._render(*args, **kwargs) │····································································· File "/tmp/tmpQunSot/backend_3c11be5cd8f902046f5cde104dcea579.py", line 385, in render │····································································· slot_middle(__stream, econtext.copy(), rcontext) │····································································· File "/tmp/tmphJej5b/detail_d0f015b8b66c6a3c182f77a550fae4a7.py", line 19077, in fill_middle │····································································· __condition = (not (_lookup_attr(getitem('member'), 'membership_date') > date(2018, 11, 31)) and ((_lookup_attr(getitem('member'),│····································································· 'membership_loss_date') is None) or (_lookup_attr(getitem('member'), 'membership_loss_date') >= date(2018, 11, 1)))) │····································································· ValueError: day is out of range for month │····································································· │·····································································

ari-pep commented 6 years ago

fixed, but the new columns we added are not yet shown

buffstop commented 6 years ago

The issue persists here (in production I get the above mentione error, in test I get "Internal Server Error").

ari-pep commented 6 years ago

sorry, ment to write this on another issue :/

ari-pep commented 6 years ago

with current staging, this happens when one of the balance columns has NULL, which is the case for members added before the relevant db migration I think. Which is not the case for anyone in production.

this could be addressed when switching to generalized transactions

buffstop commented 6 years ago

Production is affected. Tested: myself: https://join.pep.coop/detail/5

ari-pep commented 6 years ago

the links work now, but the column is misleading: leads to membership detail, not the shares. This maybe was meant for /shares_detail/$id but the id indicates share package not member id. upstream has the same behaviour :/