Closed
Conversation
…model Domain model (12 tables): Grade → ContentModule → Topic → Lesson, with LessonPlan, ProblemSet, ExitTicket, Homework as separate child tables. Assessment at module level. Problem (polymorphic source), Standard, and StandardTagging for cross-cutting concerns. Content pipeline: Aspose DOCX→HTML normalizer (Nokogiri-based whitespace cleanup), HTML parser that splits lessons into components by detecting Name/Date student section boundaries, and importer that walks the Grade 5 directory tree. Full round-trip validation passing 149/149 lessons. Browse UI: Grade → Module → Topic → Lesson navigation with Tailwind. Lesson show page links to reconstructed full HTML and individual components. PDF rendering via ferrum (Chrome headless) and WeasyPrint exploration. Key finding: Aspose HTML is source data, not presentation layer. Browser rendering differs from Word's typesetting (~15% tighter text in Chrome). Web templates should be purpose-built, not retrofitted from Word CSS. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…dling Wraps Plurimath with fixes for EngageNY-specific OMML patterns: - Strip Word track change markup (w:ins/w:del) from equations - Convert Wingdings/Symbol font references (w:sym) to Unicode Maintains a manifest of where fixes are applied so humans can prioritize second-looks at affected content. Currently converts 14,375 of 14,377 Grade 5 OMML expressions (99.99%). Also switches Plurimath to jcasimir/plurimath fork which adds support for accent elements (m:acc) with namespaced attributes, fixing 253 geometry overbar expressions. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Expression holds every representation of a math expression: OMML (Word source), MathML (web-ready), text representation, and conversion status for tracking which need human review. ExpressionExtractor reads DOCXs from the EngageNY archive, extracts all <m:oMath> elements, converts via MathConverter, and stores as Expression records on their source Lesson. Grade 5 results: 13,660 expressions extracted across 124 lessons, 13,659 converted to MathML (99.99%), 1 unconverted (Wingdings sym in complex fraction expression — tracked in manifest). Run with: bin/rails engageny:extract_math Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add khan_academy source and khan_practice resource type to SupplementalResource - Add SupplementalResourcesHelper with display filtering, labels, and YouTube embed support - Update lesson views to show embedded YouTube videos and supplemental resource links - Update topic views to show topic-level resources (quizzes, newsletters, Khan Academy) - Eager-load supplemental_resources in lessons and topics controllers - Add CCSS Math standards data for Grades 4-6 (110 standards from SirFizX/standards-data) - Add Khan Academy EngageNY Grade 5 alignment data (6 modules, 36 topics) - Add YouTube video metadata for 135 EMBARC lesson videos (titles, thumbnails via oEmbed) - Add Ruby scripts: fetch_ccss_standards.rb, fetch_khan_alignment.rb, fetch_video_metadata.rb - Add rake tasks: ccss:import, khan:import, khan:clear Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…frastructure Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Bumps [thruster](https://github.com/basecamp/thruster) from 0.1.19 to 0.1.20. - [Changelog](https://github.com/basecamp/thruster/blob/v0.1.20/CHANGELOG.md) - [Commits](basecamp/thruster@v0.1.19...v0.1.20) --- updated-dependencies: - dependency-name: thruster dependency-version: 0.1.20 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
bot
added
dependencies
Author
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
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.
Bumps thruster from 0.1.19 to 0.1.20.
Changelog
Sourced from thruster's changelog.
Commits
5c9dce8Bump versionbddb1d8Merge pull request #120 from basecamp/check-tls-domain-for-redirects186b8f5Only redirect HTTP->HTTPS for policy-allowed hostsd925847Address PR review feedback from Copilotb090a2eValidate Host header against TLS domains in HTTP→HTTPS redirectb47fc5bci: harden GitHub Actions workflows (#117)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)