microsoft / xblock-azure-media-services

xblock-azure-media-services
Other
14 stars 19 forks source link

Implement initial Azure Media Service REST API communication #39

Closed oksana-slu closed 6 years ago

oksana-slu commented 6 years ago

@kabirkhan @saravpal

  1. Extended native OpenEdx Organization model with SettingsAzureOrganization model;
  2. Configured admin section for SettingsAzureOrganization model fields;
  3. Created client to communicate with Azure Media Service API - MediaServicesManagementClient;
  4. Implemented xBlock studio view to customize editing modal window behavior;
  5. Switched from Mako to Django template engine;
  6. Configured initial CircleCI service automatic checks;
  7. New code base covered with unit tests.

AMS content is shared per OpenEdx organization

Technically AMS API credentials are configured per Organization. If there is no explicit configuration for the current organization - fallback to global (per OpenEdx installation) configuration is performed. To configure global AMS API ones should add to lms.env.json into FEATURES section following settings:

"AZURE_CLIENT_ID": <azure-open-directory-app-id>
"AZURE_CLIENT_SECRET": <azure-open-directory-app-secret-key>
"AZURE_TENANT": <azure-open-directory-id>
"AZURE_REST_API_ENDPOINT": <azure-media-service-rest-api-endpoint>

CircleCI service automatic checks include:

To be able to run all checks manually ones should:

NOTE: assuming Ubuntu as a testing environment OS

msftclas commented 6 years ago

CLA assistant check
All CLA requirements met.

sdolenc commented 6 years ago

I apologize it has taken me so long to review this. The PR was submitted while I was out on vacation so I missed it, but I should have time to review it this afternoon update:

kabirkhan commented 6 years ago

Awesome initial work!