feat(configuration): adds MySQL and SSO to integrate with Open edX to prototype course-based permissions #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
so that Superset can access the Open edX databases in the Tutor dev environment.
by adding the required dependencies and a set of example code and configuration which connect to the Tutor dev environment.
and documentation on how to connect to the Open edX MySQL database.
* is_superuser/is_staff fetched directly from configured Open edX database connection * course membership fetched from Open edX API * jinja custom template function fetches course membership and returns appropriate WHERE clause to filter by course_id
pomegranited
force-pushed
the
jill/openedx-sso
branch
from
093e9a8
to
752c509
Compare
8 tasks
(Users without groups cannot see any charts or datasets.)
In order to include the current user in the query cache key, the `current_username` macro needs to be used in the query. This change fixes our `can_view_courses` macro to require a username, so that the username will be used in the cache key.
pomegranited
force-pushed
the
jill/openedx-sso
branch
from
ddc5fac
to
ecfa249
Compare
|
Superseded by openedx-unsupported/tutor-contrib-superset#1 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
SUMMARY
Proof of concept to test whether Superset can be configured to restrict access to data by course membership.
See openedx/wg-data#21
TESTING INSTRUCTIONS
See added README-openedx.md for full instructions.