search

pinax / symposion

a Django project for conference websites
BSD 3-Clause "New" or "Revised" License
299 stars 146 forks source link

Adds a schedule_json view #62

Closed codersquid closed 10 years ago

codersquid commented 10 years ago

This is based off of @taavi's pull request, #45, and is also informed from views.py in the pyohio/pyohio repo.

Testing: It wasn't clear how to test symposion on its own, and I kludged something together so that I could test this change outside of the context of a conference site. To test the change, make sure that requirements-test.txt are installed, then run ./runtests.py from within the schedule app test directory. The requirements-text.txt, factories.py, runtests.py, and test_views.py are all related to testing the change. Building up a nice schedule factory is a pain, so I'm stopping now to make this PR to get comments. I'm willing to bet people with more experience than me can provide guidance here.

codersquid commented 10 years ago

We've had no review comments for this. Considering that no one is using this functionality yet, I'm willing to risk checking it in with only the current test coverage.