Skip to content

docs: xblock aside developer docs - #1507

Open
asadali145 wants to merge 3 commits into
openedx:sarina/asidesfrom
mitodl:asad/update-aside-docs
Open

docs: xblock aside developer docs#1507
asadali145 wants to merge 3 commits into
openedx:sarina/asidesfrom
mitodl:asad/update-aside-docs

Conversation

@asadali145

Copy link
Copy Markdown

Update developer docs for xblock asides.

@openedx-webhooks openedx-webhooks added the open-source-contribution PR author is not from Axim or 2U label Aug 3, 2026
@openedx-webhooks

openedx-webhooks commented Aug 3, 2026

Copy link
Copy Markdown

Thanks for the pull request, @asadali145!

This repository is currently maintained by @openedx/wg-maintenance-docs.openedx.org.

Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review.

🔘 Get product approval

If you haven't already, check this list to see if your contribution needs to go through the product review process.

  • If it does, you'll need to submit a product proposal for your contribution, and have it reviewed by the Product Working Group.
    • This process (including the steps you'll need to take) is documented here.
  • If it doesn't, simply proceed with the next step.
🔘 Provide context

To help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:

  • Dependencies

    This PR must be merged before / after / at the same time as ...

  • Blockers

    This PR is waiting for OEP-1234 to be accepted.

  • Timeline information

    This PR must be merged by XX date because ...

  • Partner information

    This is for a course on edx.org.

  • Supporting documentation
  • Relevant Open edX discussion forum threads
🔘 Get a green build

If one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green.

Details
Where can I find more information?

If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources:

When can I expect my changes to be merged?

Our goal is to get community contributions seen and reviewed as efficiently as possible.

However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:

  • The size and impact of the changes that it introduces
  • The need for product review
  • Maintenance status of the parent repository

💡 As a result it may take up to several weeks or months to complete a review and merge your PR.

@asadali145

Copy link
Copy Markdown
Author

@sarina I have this PR on top of your aside docs PR. Right now, I have created it against your PR to see the updates.

I have verified all updates in this PR. I need to verify the sample XBlock aside examples from the PR.

Should we create this as a PR against your branch or the main branch?

@asadali145
asadali145 force-pushed the asad/update-aside-docs branch 3 times, most recently from 947f303 to 319e3a7 Compare August 3, 2026 12:45
@mphilbrick211 mphilbrick211 moved this from Needs Triage to Waiting on Author in Contributions Aug 3, 2026
@asadali145
asadali145 force-pushed the asad/update-aside-docs branch from 319e3a7 to 083c7fb Compare August 3, 2026 14:22
@asadali145
asadali145 marked this pull request as ready for review August 3, 2026 15:16
@asadali145

Copy link
Copy Markdown
Author

@sarina, I have reviewed all of the changes, including the base changes from your PR plus the updates in this PR. I have also tested the sample asides, and they look good.

Please let me know if we want to create this PR against main and then we can proceed to a final review from Dave maybe.

@sarina sarina left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a couple questions here, once we work those out we can point this PR at main and merge the whole thing in.

Comment thread source/developers/concepts/about_xblock_asides.rst Outdated

The Authoring micro-frontend has no native code for rendering or
toggling Asides — it does not know Asides exist. What it does have is
a unit editor that embeds the legacy Studio unit page in an iframe, so

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Interesting, I think I'm not following here though. What do you need to do, as a course author), to look at the legacy Studio unit page to see the Asides render?

Expanding on this, will the deprecation of legacy Studio break this flow entirely? And if you can only see Asides in legacy Studio, how do things render in the learning MFE?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

First of all, we should understand that AUTHOR_VIEW and STUDIO_VIEW are 2 different things that can serve an identical UI/View, but it is rendered in different places in legacy Studio. So, I used AUTHOR_VIEW or STUDIO_VIEW in the usage.

What do you need to do, as a course author, to look at the legacy Studio unit page to see the Asides render?

If you are using legacy Studio, and an aside supports both AUTHOR_VIEW and STUDIO_VIEW, you should see it render in 2 places:

If you are using the new authoring MFE:

  • Open a course
  • Open a unit page with a block that the aside supports
  • You should see AUTHOR_VIEW rendered with the block contents at the end of the block. This is the same as above for the legacy unit page.

will the deprecation of legacy Studio break this flow entirely?

No, it will not break because the new authoring MFE uses an iframe to render the unit page, and it still supports the AUTHOR_VIEW, and we use it in the MITODL-maintained asides.

And if you can only see Asides in legacy Studio, how do things render in the learning MFE?

learning MFE displays STUDENT_VIEW of an aside so it will display whatever the Xblock Aside's STUDENT_VIEW returns. Regardless of its support in authoring MFE.

MORE DETAILS

Let me simplify all this for you. We have 2 studio views/UIs: legacy and new authoring MFE.

  • We have 2 aside views, AUTHOR_VIEW and STUDIO_VIEW, in an aside's code. The main purpose of these 2 is to configure the Aside data if needed, like enable/disable.
  • Legacy studio UI:
    • Legacy studio supports both AUTHOR_VIEW and STUDIO_VIEW of asides. Author and Studio views render in 2 different places.
    • AUTHOR_VIEW renders on the unit page; this is supported in both legacy and new authoring MFE. The unit page is basically an IFrame served by the studio backend. It has no connection to Authoring MFE, apart from the fact that it just renders the Iframe served by the studio backend.
    • STUDIO_VIEW renders at:
      • Click the edit button on a block, like a problem block.
      • In the editing dialog UI, there should be Editor, Settings, and Plugins in the title bar. Click "Plugins".
      • You should see the STUDIO_VIEW rendered here. MITODL's legacy repo for rapid response aside, has some details https://github.com/mitodl/rapid-response-xblock/tree/master. NOTE: This is a deprecated repo referenced here for clarity in Studio View.
  • Authoring MFE / new Studio UI:
  • Learning MFE
    • Learning MFE displays the STUDENT_VIEW of an aside, so it does not depend on AUTHOR_VIEW or STUDIO_VIEW.

Let me know if you need more help understanding this. We can even have a meeting to clarify the understanding.

Comment thread source/developers/concepts/about_xblock_asides.rst
asadali145 and others added 2 commits August 12, 2026 14:32
Co-authored-by: Sarina Canelake <sarina@axim.org>
Add suggestion from code review: the limitations section is 200+
lines into the document, so surface it earlier with a warning
directive near the top that links to it.
@asadali145
asadali145 requested a review from sarina August 12, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

open-source-contribution PR author is not from Axim or 2U

Projects

Status: Waiting on Author

Development

Successfully merging this pull request may close these issues.

4 participants