IMS fixed their validator to check for the correct
XSD documents. I made some minor changes so that
everything will validate
refs #3396
Change-Id: I8337c24e7aa979bdeaf4b533bd86667336d1c49f
Reviewed-on: https://gerrit.instructure.com/3040
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
This adds calculated/numeric questions to the common cartridge export
The assignment for graded quizzes will also get exported now
refs #3396
Change-Id: Iac9d5c18f57f3cb89a56d20e9320403ac1495356
Reviewed-on: https://gerrit.instructure.com/3038
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
This adds exporting for matching, multiple dropdowns,
missing word, and fill in multiple blanks
refs #3396
Change-Id: Ie7dc75533e35a74137ff9a514cc6ca8b211aa3d7
Reviewed-on: https://gerrit.instructure.com/2926
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
All of the non-CC-supported quiz question types
still need to be exported
refs #3396
Change-Id: I0a8225a402577994006e424189515e2b432a0d9d
Reviewed-on: https://gerrit.instructure.com/2924
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
also did a little refacter of the course settings to
make a place to put all the canvas-specific data
refs #3396
Change-Id: Ibc12b2939439738ef6525f51b5e783e9e9bca68b
Reviewed-on: https://gerrit.instructure.com/2919
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
This is needed for exporting full courses
refs #3396
Change-Id: Ie90714ca35c5f5733be2a2a132fe33f90cfed0d2
Reviewed-on: https://gerrit.instructure.com/2916
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
This uses the defined discussion topic format for
CC, but references an additional xml file with
canvas-specific properties. These can also have
an embedded assignment
refs: #3396
Change-Id: Ibf33a7509b529e2a53a3d1a511aa251b013c7639
Reviewed-on: https://gerrit.instructure.com/2915
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
This allows non-canvas systems to import the syllabus
html without touching the proprietary settings file
refs #3396
Change-Id: I12b49e2ad7e2f7014acab43795c084b674af8dac
Reviewed-on: https://gerrit.instructure.com/2914
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
This creates a manifest file and course settings document
and also started an .xsd for canvas-specific features
refs #3396
Change-Id: If8d11ba7a613db20373ba4cfc9c765eb9ef10b25
Reviewed-on: https://gerrit.instructure.com/2911
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
Enable auto_flush on the rails logging, so that the log is flushed to
disk after each log line, rather than each 1000 log lines.
Also redirect stdout and stderr if we've daemonized.
Change-Id: Iea89db81b4ac29a63614eff37da9216d6346a6f2
Reviewed-on: https://gerrit.instructure.com/2895
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
Reviewed-by: JT Olds <jt@instructure.com>
refs #4052
This only happens when using the redis queue for page views.
Also avoid a mass assignment warning being logged.
Change-Id: Id8b38ecc59dce41987c3fe9ccdac7a311012220e
Reviewed-on: https://gerrit.instructure.com/2886
Reviewed-by: Zach Wily <zach@instructure.com>
Tested-by: Hudson <hudson@instructure.com>
We made generating the recipient list a delayed job, but the broadcast_policy
stuff runs asynchronously in a request, so it thought there was no one to send
a notification to. Make the recipient list synchronous for now to fix the bug
- in the future we may want to make the notification part async too, along
with generating the recipient list.
Change-Id: Ib2648135c228c1531d446a74ceb58f82cf4b5ab1
Reviewed-on: https://gerrit.instructure.com/2881
Reviewed-by: Brian Palmer <brianp@instructure.com>
Tested-by: Hudson <hudson@instructure.com>
Teachers can specify that before students see any
messages in a topic they need to post at least once
in the topic themselves. This is used for
assignment topics or to help encourage students
to think for themselves instead of just posting
"what he said".
fixes#3871
Change-Id: I946b71ec47bfe135f06faff68598c6f19c088a27
Reviewed-on: https://gerrit.instructure.com/2370
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
The link in the gradebook to see information should link to the
speed grader instead of to the submission grades page, which has
less information.
fixes#4115
Change-Id: I6e9beae2d0e4ead7f0da0e025dcc5137d98893d3
Reviewed-on: https://gerrit.instructure.com/2837
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
the problem was that it was using a global js variable
'id_maps' to know which student was on which row.
when I removed rows not in this section, the rows
would be off.
Change-Id: Ifb4813cae5074c351a16767e1bda4dbaf2092b61
Reviewed-on: https://gerrit.instructure.com/2856
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Zach Wily <zach@instructure.com>
closes#3958
This will be pretty slow for somebody who is a teacher in a lot of
courses, but I guess it's not any worse than the dashboard. We might
want to cache this eventually.
Change-Id: I26d68bc285fba0747cdc960a67b75e0329acb6cf
Reviewed-on: https://gerrit.instructure.com/2864
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Zach Wily <zach@instructure.com>
unpack returns an array, so we need to get the first element from it. This
worked before, but only by accident.
Change-Id: I2f3808d85c2452b7b926294c920cbaaf9ab9f6a5
Reviewed-on: https://gerrit.instructure.com/2863
Reviewed-by: Jon Jensen <jon@instructure.com>
Tested-by: Hudson <hudson@instructure.com>
Right now they're being ordered by id. This
changes them to be alphabetical, which should
make them easier to look through.
fixes#4117
Change-Id: I341193e743823ac87880abcc140b0b13ab93d837
Reviewed-on: https://gerrit.instructure.com/2838
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Jon Jensen <jon@instructure.com>
Also did a couple other tweaks:
* Fixed the "Time" display when reloading via ajax (wasn't checking for the
right workflow_state
* Hide the "Attempts" column on quizzes that have an unlimited # of
since it doesn't contain any useful data (always "--").
Change-Id: I21182193005d7415594b9c0531c03d619a31c262
Reviewed-on: https://gerrit.instructure.com/2857
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
Going through connection_handler is the new Rails Way of doing this sort
of thing, and makes postgres happier.
Change-Id: I7f54323e68260b6901c40eeb1a7f4553f05cda4f
Reviewed-on: https://gerrit.instructure.com/2860
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Zach Wily <zach@instructure.com>
this gets us the fix in https://rspec.lighthouseapp.com/projects/5645/tickets/946
when running "rake spec" without specifying SPEC, vendor/plugins/respondus_soap_endpoint/spec_canvas/integration/respondus_endpoint_spec.rb
is the first to run. because we use transactional fixtures, whichever spec
runs first is the only one to load them, and integration tests were looking
for fixtures in test/fixures. so later tests that use fixtures (currently just
web_conferences stuff) would fail.
Change-Id: I63f6be79614afbf48301b48ca2c3d97135abfdaa
Reviewed-on: https://gerrit.instructure.com/2847
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Brian Palmer <brianp@instructure.com>
Reviewed-by: Zach Wily <zach@instructure.com>
TextHelper#truncate_html doesn't appear to have this issue, because it
always splits and truncates on word boundaries (using String#split()).
Change-Id: I8bf301055818d7080d8e12859707458ba41e89f4
Reviewed-on: https://gerrit.instructure.com/2851
Reviewed-by: Zach Wily <zach@instructure.com>
Tested-by: Hudson <hudson@instructure.com>
I also had to change the api spec stuff a bit, which caused some changes
in other specs.
Change-Id: I62e87fa4c9e79716455ca47ca342c9eec86ddc39
Reviewed-on: https://gerrit.instructure.com/2848
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Zach Wily <zach@instructure.com>
Bracken Mosbacker