Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove membership types from the client #8

Open
arkoebel opened this issue Sep 24, 2018 · 1 comment
Open

Remove membership types from the client #8

arkoebel opened this issue Sep 24, 2018 · 1 comment

Comments

@arkoebel
Copy link
Contributor

I thought that PR#6 was supposed to remove member types from the client, but maybe I was mistaken.

Membership types are still somewhat hardcoded in the client (in NewMemberCard.jsx, Upgrade.jsx, MemberTypeList.jsx).

1 - It makes merging more difficult for Worldcons who used different MemberTypes
2 - It forces to recompile the client when a new membership type needs to be added/removed

MemberTypes (and indeed anything that can change over time) should come from the API and not be hardcoded in the client.

@eemeli
Copy link
Member

eemeli commented Sep 24, 2018

#6 did help in getting rid of some of the client-side definitions, but there's certainly remnants as you've found. As is, the expectation is that you'll customise the client code directly, including these member type changes.

The current client-side architecture as a whole is far too clumsy and monolithic, and I'm working now on breaking it apart into configurable modules, here: https://github.com/maailma/kansa/tree/dev/client (note the merge back into the kansa repo, as per my comment on Gitter a bit over a week ago). As that progresses, the member types should also get sorted at some point.

FYI, I do not have current plans of backporting changes from dev to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants