-
Notifications
You must be signed in to change notification settings - Fork 40
Research insight: AI as tutor #1240
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this 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!
94c1f94 to
ff48375
Compare
|
Cloud Build time: Wed Dec 17 23:03:20 UTC 2025 |
|
This is a work in progress. Here's a list of things that still need to be updated:
|
4f8229f to
c94b589
Compare
9aef350 to
c7f2afb
Compare
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.
c7f2afb to
f64e6cc
Compare
|
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 |
davidstanke
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
A new AI research insight: AI as tutor.
This research is in conjunction with UC Berkley.
Preview URLs: