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

Travel reimbursement #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Travel reimbursement #2

wants to merge 4 commits into from

Conversation

mike2151
Copy link
Member

@mike2151 mike2151 commented Jan 7, 2018

No description provided.

@jreinstra
Copy link
Member

@mike2151 I'm not sure if you wanted me to review this, but I took a quick look and I think something important to note is our current application / hackathon / user permissions hierarchy is very overcomplicated if we are building an open source tool that other hackathons can use. I think before something like this lands, we need to do some work to simplify those models.

@ezwang It'd be great if you could help think about this also. Essentially I think we should change it so that the application questions + answers are hardcoded into the models rather than dynamically created in a database. The dynamic creation is great if we were creating a SaaS management tool like initially intended but will make life a lot harder to do stuff like this. Since we're deciding to do open source, we should re-architect the system a bit before adding features. Let me know if I can help with this, although I want to let you guys take charge of this to the extent you're able to.

@ezwang
Copy link
Contributor

ezwang commented Jan 11, 2018

Perhaps we could have the common fields hardcoded (contact information, resume, GitHub, etc) and leave Hackathon directors with the option to add a few extra custom free response fields? This way we can preserve some of the flexibility while making the application easier to develop.

@jreinstra
Copy link
Member

@ezwang that seems reasonable to me - sorry that im just seeing this - my github notifications don't work very well

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

Successfully merging this pull request may close these issues.

3 participants