Skip to content

Add overview lectures to website (closes #18) - #19

Merged
mmcky merged 2 commits into
mainfrom
overview-lectures
May 22, 2026
Merged

Add overview lectures to website (closes #18)#19
mmcky merged 2 commits into
mainfrom
overview-lectures

Conversation

@mmcky

@mmcky mmcky commented May 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds an Overview Lectures section to website/index.html, placed between About and Table of Contents — reflects John's note in Add overview lectures to the website #18 that the two lectures are an overview of the whole project, not paired with a specific volume.
  • Mirrors lecture1.pdf and lecture2.pdf into pdf/ as build artifacts (initial copy from book-dp2/presentation/).
  • Follow-up issue to be filed on book-dp2 to wire these into the existing sync.yml so future updates propagate automatically, and to make a final call on source-of-truth.

Resolves the interim placement question raised in #18 without waiting for the broader MyST migration / website re-jig.

Test plan

  • Render website/index.html locally and confirm the Overview Lectures section appears below About
  • Both lecture PDF buttons resolve once the branch is merged to main (raw GitHub URLs become live)
  • No regression to existing Vol I / Vol II tabs or Download buttons

Closes #18

🤖 Generated with Claude Code

Adds an "Overview Lectures" section to the landing page linking the two
high-level lecture PDFs that cover both volumes. PDFs are mirrored from
book-dp2/presentation/ as build artifacts; long-term source-of-truth is
tracked in book-dp2.

Closes #18

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 22, 2026 04:35
@netlify

netlify Bot commented May 22, 2026

Copy link
Copy Markdown

Deploy Preview for resonant-pie-f0089d ready!

Name Link
🔨 Latest commit a333f47
🔍 Latest deploy log https://app.netlify.com/projects/resonant-pie-f0089d/deploys/6a0fe0f9d7d2b20008238324
😎 Deploy Preview https://deploy-preview-19--resonant-pie-f0089d.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds an “Overview Lectures” section to the landing page so the two project-wide lecture PDFs are discoverable in a dedicated spot (between About and Table of Contents), consistent with the intent described in issue #18.

Changes:

  • Adds a new “Overview Lectures” section on website/index.html.
  • Adds two buttons linking to lecture1.pdf and lecture2.pdf under pdf/ via the repo’s existing GitHub raw URL pattern.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Per review on #19: section reads better between ToC and Download
than above ToC.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
github-actions Bot temporarily deployed to commit May 22, 2026 04:52 Inactive
@mmcky

mmcky commented May 22, 2026

Copy link
Copy Markdown
Contributor Author

@jstac this PR adds a section below the Table of Contents and above Downloads.

Screenshot 2026-05-22 at 2 53 55 pm

@mmcky

mmcky commented May 22, 2026

Copy link
Copy Markdown
Contributor Author

@jstac I have opened https://github.com/QuantEcon/book-dp2/issues/153 to fix the auto-sync from book-dp2.

Alternatively, would you like to move the presentations here?

I think not -- sync -- because all other sources of truth are based in book-dp2 and book-dp1

@jstac

jstac commented May 22, 2026

Copy link
Copy Markdown
Contributor

Thanks @mmcky , no need for autosync on those lectures.

I doubt they will change.

@mmcky
mmcky merged commit aa7cb08 into main May 22, 2026
6 checks passed
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.

Add overview lectures to the website

3 participants