docs: Updated educator docs for using Open edX as an LTI tool provider#1448
docs: Updated educator docs for using Open edX as an LTI tool provider#1448ayub02 wants to merge 6 commits intoopenedx:mainfrom
Conversation
|
@ayub02 please ping me when this is ready for review or if you want help fixing build errors. |
sarina
left a comment
There was a problem hiding this comment.
I still need to review enable_lti, index, and tpa_providers. I know it's a lot of comments, many are simply that "Open edX" has been repeatedly used as a noun when it needs to be used as an adjective.
|
|
||
| .. seealso:: | ||
|
|
||
| :ref:`Using Open edX as an LTI Tool Provider` (educator) |
There was a problem hiding this comment.
There's probably more that could be added here
There was a problem hiding this comment.
For example, the other Site Operators docs that relate to this topic.
| email already exists, the existing account is linked. This mode is useful when you need to identify | ||
| learners or associate their LTI activity with an existing Open edX account. | ||
|
|
||
| For this to work, configure your external LMS to include ``lis_person_contact_email_primary`` in |
There was a problem hiding this comment.
[clarification] who has the ability to configure the external LMS?
There was a problem hiding this comment.
This will depend on the org. But my assumption is that it'll be educators is most cases since they'll be the ones creating content in Open edX, constructing URLs and then configuring those URLs in other LMSes.
This PR updates educator and site operator docs for Open edX as an LTI provider. These updates are a result of some testing that i've conducted with Canvas along with some code review via Claude.
At present, there are 12 pages in total for both educators and site operators. This PR reduces them to 6.
Site operator docs have been generated based on feedback from @feanil and code review from Claude Code.
Docs have been generated with help from Sonnet 4.6.
Note: I've deleted the
Configuring Open edX User Authentication for LTIfile because: