Skip to content

Split reviewer media server integration#3

Open
andrewsanchez wants to merge 1 commit into
pr4289-split/reviewer-toggle-api-accessfrom
pr4289-split/mediasrv-reviewer-integration
Open

Split reviewer media server integration#3
andrewsanchez wants to merge 1 commit into
pr4289-split/reviewer-toggle-api-accessfrom
pr4289-split/mediasrv-reviewer-integration

Conversation

@andrewsanchez
Copy link
Copy Markdown

Source split from ankitects#4289.

Breakdown sections covered:

    1. Python-side NextCardData intercept
    1. Reviewer hook parity
    1. Media-serving CORS for sandboxed iframe assets
    1. External links in iframe

Source fidelity:

Follow-up notes from the breakdown:

  • Hook parity still needs a dedicated audit against the legacy reviewer.
  • External links and embedded user iframes remain an open design issue.
  • The CORS/media route behavior should be reviewed with MathJax fonts and user media.

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