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

doc: add xpert summaries overview and enabling to partner docs #2191

Merged
merged 1 commit into from
Sep 29, 2023

Conversation

ashultz0
Copy link
Contributor

@ashultz0 ashultz0 commented Sep 28, 2023

Eventually this feature may move to the shared section, but for not it is not available to open edx. course-authors builds to https://edx.readthedocs.io/projects/edx-partner-course-staff

Reviewers

  • Requester: Emily Miller

Testing

  • Ran ./run_tests.sh without warnings or errors

HTML Version (optional)

  • PDFs added here

@ashultz0 ashultz0 marked this pull request as draft September 28, 2023 17:16
To toggle Xpert Unit Summaries on or off for a specific unit
***************************************************************

#. In Studio, course team members can navigate to the **Outline** menu
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe Unit Summaries toggle won't be visible unless the Xpert Unit Summaries is turned on for the whole course. Is that right? If so, could you note that here?

@ashultz0 ashultz0 force-pushed the ashultz0/xpert-summaries branch from be08753 to 3ae071a Compare September 29, 2023 16:37
eventually this feature may move to the shared section, but for not it
is not available to open edx
@ashultz0 ashultz0 force-pushed the ashultz0/xpert-summaries branch from 3ae071a to 3639cfa Compare September 29, 2023 17:16
@ashultz0
Copy link
Contributor Author

while it builds some of the projects as part of the checks so that you can see what it's doing, I can't figure out how to get readthedocs to build the one we're actually changing here, so in lieu of that pdfs of the two main pages
overview.pdf
enabling.pdf

@ashultz0 ashultz0 marked this pull request as ready for review September 29, 2023 17:32
Copy link
Contributor

@schenedx schenedx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM

@ashultz0 ashultz0 merged commit 2241b52 into master Sep 29, 2023
1 check passed
@ashultz0 ashultz0 deleted the ashultz0/xpert-summaries branch September 29, 2023 18:30
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

Successfully merging this pull request may close these issues.

2 participants