PADV-1557 Add storage backend capabilities to SGA Xblock #7
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Tickets
Description
This PR add the feature to the SGA Xblock to be able to manage the AWS S3 buckets when Xblock settings are defined. If there are not settings, it would relay on Django's default storage.
Changes Made
How To Test
Annotations
Broken dependencies and CI process
CI process of unit testing and linting is failing due to dependencies issues. It also restrict the possibility to run it locally despite of the presence of a Tox defined workflow to do so. This is because it expects the workflows and credentails from upstream, not pearson. That could be cover in future PRs, but is not the purpose of this one so it won't be reviewed.
Lack of openedx_wrapper
As you might realize, the openedx site configurations helper is import directly into the SGA module without the use of a wrapper, which conflicts with our code quality standards. This is because most of the platform imports are doing in the same way and looking forward to port this in the upstream repository we shall not use any different approach but the one the repository has.
Upstream port
This feature is a port coming from the upstream repository
Testing cases and feature documentation would be available in next PR where we adjust the upstream feature to the site-awareness desire behavior of the Xblock