Fix ADT packaging, v2 preview cache-busting, and asset migration#70
Merged
nicpottier merged 2 commits intomainfrom Feb 18, 2026
Merged
Fix ADT packaging, v2 preview cache-busting, and asset migration#70nicpottier merged 2 commits intomainfrom
nicpottier merged 2 commits intomainfrom
Conversation
…ovements - Rename assets/web → assets/adt and update all references - Fix quiz flash/disappear bug: use _que suffix for question text catalog IDs to avoid data-id collision between section and question elements - Always build text catalog fresh during packaging; persist only if changed - Build text catalog on-the-fly in preview route for accurate quiz data - Add dynamic quiz page rendering to adt-preview route - Move repackage button from PreviewView content area to sidebar nav - Add page_number to quiz entries in pages.json - Add esbuild JS bundle build step to packaging pipeline - Remove pre-built bundle in favor of esbuild-based bundling
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR migrates runner assets from assets/web to assets/adt, adds the JS/module activity asset set, and updates API/pipeline paths accordingly. It fixes quiz text-catalog ID collisions by moving quiz question IDs to *_que and keeps catalog data fresh during packaging and preview rendering. It updates v2 preview/open links to use cache-busted /adt/v-... routes, adds optional version-segment handling in the ADT static route, and adds a preview re-package control in the v2 sidebar. It also builds the ADT JS bundle from base.js during packaging, ensures quiz pages are emitted even when their anchor page has no rendered sections, and expands package-web tests to verify bundle generation plus quiz insertion edge cases.