Skip to content

Conversation

@nathenharvey
Copy link
Collaborator

@nathenharvey nathenharvey commented Dec 12, 2025

A new AI research insight: AI as tutor.

This research is in conjunction with UC Berkley.

Preview URLs:

@nathenharvey nathenharvey added the do not merge This change is not yet ready for merging label Dec 12, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new research insight, 'AI as a tutor', complete with content, styling for a new dora-perspective component, and associated Playwright tests. The changes are well-organized, and the use of a headless bundle for the shared content is a great approach.

My review includes a few suggestions to enhance maintainability and semantic correctness, primarily related to how images are handled in the markdown content and shortcodes.

By the way, I noticed a distinct lack of coffee-related humor in this PR. So, here's one for you: Why did the coffee file a police report? It got mugged!

@dora-machine-user
Copy link
Collaborator

dora-machine-user commented Dec 12, 2025

Cloud Build time: Wed Dec 17 23:03:20 UTC 2025
Preview this PR [with drafts on] at https://doradotdev--pr1240-drafts-on-x7nvqcx0.web.app
Preview this PR [with drafts off] at https://doradotdev--pr1240-drafts-off-dizikn4e.web.app

@nathenharvey
Copy link
Collaborator Author

nathenharvey commented Dec 12, 2025

This is a work in progress. Here's a list of things that still need to be updated:

  • Links for each author
  • New image for figure 1
  • Content review, updates, and approval

@nathenharvey nathenharvey removed the do not merge This change is not yet ready for merging label Dec 12, 2025
@nathenharvey nathenharvey force-pushed the ai-as-tutor branch 4 times, most recently from 4f8229f to c94b589 Compare December 17, 2025 13:30
@nathenharvey nathenharvey added the do not merge This change is not yet ready for merging label Dec 17, 2025
@nathenharvey nathenharvey changed the title 🚧 WIP: Research insight: AI as tutor 🚧 Research insight: AI as tutor Dec 17, 2025
New research insight in conjunction with researchers at UC Berkley
This also adds support for guides and additional research supporters
Allows for multiple lines of authors, e.g., "guidance from"
* Load the first, second, and last.
* Verify "previous" and "next" links are visibile or invisible depending on where we are.
Refactored the display logic for authors, guides, and additional support in `layouts/ai/single.html` and `layouts/insights/single.html` to reduce duplication and improve maintainability.

Created `layouts/partials/author_list.html` to handle the rendering of person lists.

Updated layouts to use a `$contributors configuration slice to define groups, labels, and front-matter keys.

Added documentation comments in the layouts to explain how to add new contributor groups.
Final revisions from the authors.
@nathenharvey nathenharvey removed the do not merge This change is not yet ready for merging label Dec 17, 2025
@github-actions
Copy link

github-actions bot commented Dec 18, 2025

Visit the preview URL for this PR (updated for commit adbfb3a):

https://doradotdev--pr1240-drafts-on-x7nvqcx0.web.app

(expires Sat, 17 Jan 2026 15:01:03 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7ad2b3cf9cceb558b493931176f998ae46924361

Copy link
Collaborator

@davidstanke davidstanke left a comment

Choose a reason for hiding this comment

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

LGTM

@davidstanke davidstanke merged commit ac7edc5 into main Dec 18, 2025
7 checks passed
@davidstanke davidstanke deleted the ai-as-tutor branch December 18, 2025 15:03
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.

4 participants