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

Documentation Update #128

Open
jvjones opened this issue Jul 27, 2020 · 1 comment
Open

Documentation Update #128

jvjones opened this issue Jul 27, 2020 · 1 comment

Comments

@jvjones
Copy link

jvjones commented Jul 27, 2020

Hello,

First off, great project, looking forward to getting this fully working.

The documentation for this project could do with a slight update to include how to configure this in Studio on Open edX.

For the providers field, the provider must be supplied in the following format:
[{"name": "provider_name"}]

Just sharing this nugget as spent a few hours working that out.

@fghaas
Copy link
Contributor

fghaas commented Jul 30, 2020

@jvjones Yes you are correct. providers is a list of dictionaries, and wherever Studio expects that, you must use JSON. Which is not exactly great UX, but it's what Studio expects.

If you define multiple providers and you want to specify different capacities for them, you'd use:

[
  { "capacity": 25, 
    "name": "provider1" },
  { "capacity": 50, 
    "name": "provider2" }
]

Also, note that it's a JSON, not a Python list. Thus:

[
  { "capacity": 25, 
    "name": "provider1" },
  { "capacity": 50, 
    "name": "provider2" },
]

... would actually cause an error, as a trailing comma in a list is not allowed in JSON.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants