Skip to content
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

Contribute: Add proper header for export course download #560

Open
OmarIthawi opened this issue Apr 6, 2020 · 2 comments
Open

Contribute: Add proper header for export course download #560

OmarIthawi opened this issue Apr 6, 2020 · 2 comments
Assignees
Labels
upstream-contribution A suggestion for contributing changes to Open edX

Comments

@OmarIthawi
Copy link

#559 seems like a good fix to contribute upstream.

@thraxil
Copy link

thraxil commented Apr 8, 2020

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
Copy link
Author

Thanks Anders for the info.

@OmarIthawi OmarIthawi added the upstream-contribution A suggestion for contributing changes to Open edX label Sep 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream-contribution A suggestion for contributing changes to Open edX
Projects
None yet
Development

No branches or pull requests

3 participants