-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add ability to upload imscc files #9
Conversation
Create a course Upload content and process Add specs
No, because it isn't one of the models for the canvas structure. CanvasCourse doesn't make sense to me to be in there for that reason. It's not really a model. |
Makes sense. You can go ahead an merge then. Thanks James! |
Will you pull the branch down and test it too? |
The specs are passing, I do get a 302, which makes rest-client throw an exception, when it is trying to upload the package. I think I may not have something set up quite right. Did you run into that at all? This is where it is trying to make the post request to |
I don't get that exception. Which was weird because canvas said it would happen but it would never happen to me. The docs say it should send back a 302 and I would need to post to the url it came back with in the header as the "location" for it to start processing. But for me it always just start processing... So I need to dig in a little more it looks like. |
Or are you talking about in the specs you get that? |
Nope, the specs are fine. It is the post request that is being made by |
Looks good to me. |
Create a course
Upload content and process
Add specs