search

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

Contribute: Add proper header for export course download #560

Open OmarIthawi opened 4 years ago

OmarIthawi commented 4 years ago

https://github.com/appsembler/edx-platform/pull/559 seems like a good fix to contribute upstream.

thraxil commented 4 years ago

Agreed, although it would have to be part of a larger patch to add boto3 support since they're still using the old version: https://github.com/edx/edx-platform/blob/master/cms/djangoapps/contentstore/views/import_export.py#L376

They are already on django-storages 1.8, so we don't need to include #548 but it will need #549 #550 and #559

OmarIthawi commented 4 years ago

Thanks Anders for the info.