search

mitodl / ccxcon

CCXCon API
GNU Affero General Public License v3.0
7 stars 0 forks source link

Create CCX view should send a JSON encoded payload #136

Closed giocalitri closed 8 years ago

giocalitri commented 8 years ago

What are the relevant tickets?

fixes #134

What's this PR do?

Fixes a bug in the creation of the CCX from CCXCon when a list of modules was passed

Where should the reviewer start?

views.py

How should this be manually tested?

Use the client script to create a CCX and pass some chapters modules.

Any background context you want to provide?

You need an EDX instance configured to work with CCXCon

What gif best describes this PR or how it makes you feel?

giocalitri commented 8 years ago

@noisecapella done

giocalitri commented 8 years ago

@noisecapella done again

noisecapella commented 8 years ago

I think you missed my comment on renaming the variable :)

giocalitri commented 8 years ago

yeah, github had not updated the comment before I pushed. It should be fine now

noisecapella commented 8 years ago

:+1: