appsembler / edx-platform

Appsembler Tahoe fork of Open edX. Branches: `main` for Juniper and `hawthorn/main` for Hawthorn.
https://appsembler.com/products/tahoe/
GNU Affero General Public License v3.0
13 stars 15 forks source link

grades download reports from S3 getting bad links #38

Closed bryanlandia closed 6 years ago

bryanlandia commented 8 years ago

Test Drive user reported not being able to download CSV grade reports from http://lms.edxdemo.appsembler.com/courses/course-v1:TheSoftwareGuild+SG101+2016_Summer/instructor#view-data_download

and I can confirm. Links are like:

On prod AMC in django shell, I did:

from django.conf import settings
settings.GRADES_DOWNLOAD
{u'STORAGE_TYPE': u'S3', u'ROOT_PATH': u'grades', u'BUCKET': u'edx-amcalpha-grade-bucket'}

Not sure why the method on the S3ReportStore for returning links is returning file:/// links but it is. Configuration looks correct.

https://github.com/appsembler/edx-platform/blob/appsembler/amc/testdrive/lms/djangoapps/instructor_task/models.py#L316

bryanlandia commented 8 years ago

I'm not positive this is specific only to the appsembler/amc/testdrive branch.

bezidejni commented 8 years ago

@bryanlandia That's not the Testdrive server, that's Dogwood Demo Server. The config for that is here: https://github.com/noderabbit-team/edx-configs/blob/master/dogwood_demo/production/files/server-vars.yml

natea commented 8 years ago

@bryanlandia @bezidejni on a demo with a prospect today, this bug reared its ugly head. Can we get this fixed on both the Dogwood Demo server and the Appsembler Academy site, so that we can properly demo the grade download feature?

tkeemon commented 6 years ago

Closing because I don't think any of those servers are in use anymore.