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

Colleges should be configurable #51

Open
finbargiusti opened this issue Jul 26, 2018 · 4 comments
Open

Colleges should be configurable #51

finbargiusti opened this issue Jul 26, 2018 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@finbargiusti
Copy link
Contributor

Problem
As OSS, it is standard to de-localize the software. I understand the purpose of universale is to serve 5 colleges, and that shouldn't change, but for those forking this project or running it for colleges in their area, college options and others should be easily modifiable.

Solution
An object should be added to Django/universale/settings that would be referenced when building the user forms. Also, to support longer lists of universities, a search dropdown system should be added. (example)

I could work on this, however I'm busy the next few days (curr. on holidays in Mexico) so if someone gets there before me, that would benefit universale.

Thanks for reading and good luck 😃

@vedantpuri
Copy link
Owner

Will try to see! I am busy working on a few other things as well but will try to get on it!

@vedantpuri
Copy link
Owner

@immohsin Has created a separate file for the variable colleges. I think this could be useful for what we are trying to achieve. Just thought would notify you of this change !

@vedantpuri vedantpuri added the bug Something isn't working label Jul 30, 2018
@finbargiusti
Copy link
Contributor Author

#52 Looks good, however I think maybe in the future we could add a global options file for things people might want to change before mirroring / editing this service's SC @vedantpuri

@vedantpuri
Copy link
Owner

Yeah that makes sense. That could be done in choices.py itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants