Converted to portable app; removed site-specific components #3
Conversation
…ake the app portable. Also removed settings.py, and non-app-specific urls in urls.py
|
I've tried to cherry pick this in my fork, but as you removed the manage.py and base.html, it can't work as a standalone anymore. |
|
Wow. This was a long time ago. But if I remember rightly, I was trying to make it into an embedded app instead of a standalone one. On second thoughts, you probably shouldn't be merging it here in that case? |
|
That's right. I want to use it as a portable app too, but I also still wanted to test in standalone before integrating (in order to see how easy it is to create a survey, add question etc.). I think adding those files permit to do both : Pierre-Sassoulas@82bcb7a |
|
Then I guess we should keep the portable package, and have a dummy app as well for testing. So when someone wants to deploy it, they can copy just the portable part (preferably in a separate folder) and leave the rest. Is it possible to merge it in but add |
|
I think its what I've done in Pierre-Sassoulas@82bcb7a. (apply your pull request, then add base.html, manage.py and settings.py back outside of the survey directory). I did not understand what you meant by "having a dummy app for testing" so maybe not. Feel free to tell me what you meant. |
Rearranged directory structure and made urls and templates more generic, to make the app portable.
Also removed settings.py, and non-app-specific urls in urls.py.
This is a major change; I don't know if it's what the app was originally meant for, but I was doing it anyway for my project and thought it may be useful to others. If it doesn't fit then maybe it could be made into a separate project...?
Note: I also removed the four required survey fields (interviewer, interviewee, conditions, comments) because they didn't work for me: sorry, I know I should have done that in a separate commit, but I'm new to git so still haven't fully got the hang of it :(