Add visual pitch deck slides: problem/solution breakdown, competitor comparison, architecture - #59
Add visual pitch deck slides: problem/solution breakdown, competitor comparison, architecture#59Tar-ive wants to merge 4 commits into
Conversation
…comparison, architecture - Add 8 new slides with AI-generated diagrams and competitor screenshots - Problem visual: 3 pain points (not knowing taste, finding gifts, price hunting) - Solution visual: how Giftmaxxing solves each (swipe, Maxi catalog, price tracking) - Competitor screenshots: side-by-side Elfster, Gifts.com, Giftmaxxing, Giftpack, MyRegistry - Competitor flows: step-by-step comparison of how each platform works vs us - Pipeline diagram: 5-stage taste signal → recommendation pipeline - Taste engine: multiple signals converging into Maxi for recommendations - Architecture: Frontend/Intelligence Layer/Infrastructure breakdown - Remove redundant text-only problem, solution, product, why-now slides per user feedback - Reorder: competitor comparison before pipeline/taste engine slides - Fix slide overflow: constrain images to max-height for proper viewport fit Co-Authored-By: Saksham <tarive22@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughThe deck swaps in centered-image slides for problem and solution, removes the product section, normalizes later footer labels, and appends new image-backed slides after Competition. ChangesDeck slide flow refresh
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
deck/index.html (1)
340-341: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick winRenumber or drop the hard-coded footer indices.
Deleting the product slide and appending five new sections changed the deck order, but the static footer labels now drift from the actual sequence (
Market · 08,Business model · 09,Competition · 10,Architecture · 15). That creates numbering skew (index mismatch) alongside Reveal’s livec/tcounter.Also applies to: 484-600
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@deck/index.html` around lines 340 - 341, The hard-coded footer slide numbers are now out of sync with the deck order because the slide sequence changed, so update or remove the static indices in the footer labels to match the current presentation flow. Adjust the relevant slide sections in the deck markup where the footer text is defined so labels like Market, Business model, Competition, and Architecture reflect the real order, or rely on the live Reveal counter instead of fixed numbers.
🧹 Nitpick comments (1)
deck/index.html (1)
247-253: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winExtract the repeated image-slide chrome into shared classes.
The same flexbox wrapper and framed-image styling is duplicated across seven slides, and the height cap is already split between
420pxand380px. A shared class pair like.visual-slide/.visual-slide--compactwould make viewport tuning a single edit instead of seven.Also applies to: 289-295, 486-492, 516-518, 540-542, 564-566, 588-590
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@deck/index.html` around lines 247 - 253, The repeated image-slide layout is duplicated across multiple slide blocks, so extract the shared flex wrapper and framed image styling into reusable classes in the deck markup/styles. Update the affected slide sections that currently use the same wrapper/image attributes to use a common class pair such as visual-slide and visual-slide--compact, and keep the height cap centralized so viewport tuning only needs one edit.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@deck/index.html`:
- Around line 582-586: The pricing claim in the lead paragraph is missing a
verification qualifier, so update the copy in the slide content that mentions
the “≈ $0.60 per 10k gift ideas” cost to include a source/date assumption or a
brief “verify before investor use” note, consistent with the Market and Business
Model slides. Keep the change localized to this descriptive text so the claim is
clearly labeled without altering the surrounding layout.
---
Outside diff comments:
In `@deck/index.html`:
- Around line 340-341: The hard-coded footer slide numbers are now out of sync
with the deck order because the slide sequence changed, so update or remove the
static indices in the footer labels to match the current presentation flow.
Adjust the relevant slide sections in the deck markup where the footer text is
defined so labels like Market, Business model, Competition, and Architecture
reflect the real order, or rely on the live Reveal counter instead of fixed
numbers.
---
Nitpick comments:
In `@deck/index.html`:
- Around line 247-253: The repeated image-slide layout is duplicated across
multiple slide blocks, so extract the shared flex wrapper and framed image
styling into reusable classes in the deck markup/styles. Update the affected
slide sections that currently use the same wrapper/image attributes to use a
common class pair such as visual-slide and visual-slide--compact, and keep the
height cap centralized so viewport tuning only needs one edit.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: b46f0001-85b3-44f9-8564-c9cdc593eed5
⛔ Files ignored due to path filters (17)
deck/slides/architecture.pngis excluded by!**/*.pngdeck/slides/competitor-comparison.pngis excluded by!**/*.pngdeck/slides/competitor-flows.pngis excluded by!**/*.pngdeck/slides/elfster-how-it-works.pngis excluded by!**/*.pngdeck/slides/elfster-shop.pngis excluded by!**/*.pngdeck/slides/elfster.pngis excluded by!**/*.pngdeck/slides/feature-matrix.pngis excluded by!**/*.pngdeck/slides/giftmaxxing-features.pngis excluded by!**/*.pngdeck/slides/giftmaxxing-hero.pngis excluded by!**/*.pngdeck/slides/giftpack.pngis excluded by!**/*.pngdeck/slides/gifts-com.pngis excluded by!**/*.pngdeck/slides/myregistry.pngis excluded by!**/*.pngdeck/slides/pipeline-diagram.pngis excluded by!**/*.pngdeck/slides/problem-breakdown.pngis excluded by!**/*.pngdeck/slides/real-competitor-comparison.pngis excluded by!**/*.pngdeck/slides/solution-breakdown.pngis excluded by!**/*.pngdeck/slides/taste-engine.pngis excluded by!**/*.png
📒 Files selected for processing (1)
deck/index.html
| <p class="lead-sm mt-s"> | ||
| Next.js 16 on Vercel up front. AWS Lambda, DynamoDB, and Bedrock Titan | ||
| Multimodal Embeddings on the back. No idle servers — the whole | ||
| recommendation pipeline costs <span class="strong">≈ $0.60 per 10k gift ideas</span>. | ||
| </p> |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Add a source/date qualifier for the per-recommendation cost claim.
This slide hard-codes ≈ $0.60 per 10k gift ideas without the kind of verification note used on the Market and Business Model slides. For pitch-deck credibility, attach the assumption date or a small “verify before investor use” qualifier here too.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@deck/index.html` around lines 582 - 586, The pricing claim in the lead
paragraph is missing a verification qualifier, so update the copy in the slide
content that mentions the “≈ $0.60 per 10k gift ideas” cost to include a
source/date assumption or a brief “verify before investor use” note, consistent
with the Market and Business Model slides. Keep the change localized to this
descriptive text so the claim is clearly labeled without altering the
surrounding layout.
There was a problem hiding this comment.
Fixed the footer numbering in 33bd46f — Market is now 06, Business model 07, Competition 08, Architecture 13 to match the actual 13-slide deck order.
Re: the cost claim qualifier — this is pre-existing content from the original deck, not introduced by this PR. Skipping as a non-functional nit. Same for the CSS class extraction suggestion.
There was a problem hiding this comment.
Skipped: comment is from another GitHub bot.
Co-Authored-By: Saksham <tarive22@gmail.com>
Co-Authored-By: Saksham <tarive22@gmail.com>
…ition slide, remove How It Works and Taste Engine slides Co-Authored-By: Saksham <tarive22@gmail.com>
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
deck/index.html (1)
453-518: 🎯 Functional Correctness | 🟠 Major | 🏗️ Heavy liftThe taste-engine slide from the PR scope is missing.
This tail section jumps from competitor flows straight to Pipeline/Architecture, and there is no dedicated taste-engine visual anywhere in the file. If that removal was intentional, the PR objective needs to change; otherwise the deck is currently missing one of the promised core visuals.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@deck/index.html` around lines 453 - 518, The deck is missing the dedicated taste-engine slide, since the sequence in the slide markup jumps from the competitor-flows section directly to pipeline and architecture. Add the missing taste-engine section in this area of the slide deck using the same structure and naming conventions as the neighboring sections (for example, data-slide attributes, wrap/eyebrow/h1/foot pattern) so the PR still includes the promised core visual.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@deck/index.html`:
- Around line 311-315: The transition slide copy in the section identified by
data-slide="transition" is out of sync with the actual slide order, since it
introduces the technical product before the following Business model and
Competition slides. Either move this transition block to immediately before the
Pipeline/Architecture content, or update the heading text in the transition
section to clearly bridge into the commercial section instead of the technical
one.
---
Outside diff comments:
In `@deck/index.html`:
- Around line 453-518: The deck is missing the dedicated taste-engine slide,
since the sequence in the slide markup jumps from the competitor-flows section
directly to pipeline and architecture. Add the missing taste-engine section in
this area of the slide deck using the same structure and naming conventions as
the neighboring sections (for example, data-slide attributes,
wrap/eyebrow/h1/foot pattern) so the PR still includes the promised core visual.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 25093ce4-7015-4a2f-bf0b-721e393df429
⛔ Files ignored due to path filters (1)
deck/slides/tam-sam-som.pngis excluded by!**/*.png
📒 Files selected for processing (1)
deck/index.html
| <!-- ════════════════ SLIDE — TRANSITION ════════════════ --> | ||
| <section data-slide="transition"> | ||
| <div class="wrap center"> | ||
| <div class="stack gap-m" style="align-items:center; text-align:center; max-width:900px; margin:0 auto;"> | ||
| <h1 class="display">Now look at the<br/><span class="grad">technical product</span>.</h1> |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Fix the transition copy/order mismatch.
Line 315 tees up the technical product, but the next slides are Business model and Competition. Either move this transition down to just before Pipeline/Architecture, or rewrite it to bridge into the commercial section.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@deck/index.html` around lines 311 - 315, The transition slide copy in the
section identified by data-slide="transition" is out of sync with the actual
slide order, since it introduces the technical product before the following
Business model and Competition slides. Either move this transition block to
immediately before the Pipeline/Architecture content, or update the heading text
in the transition section to clearly bridge into the commercial section instead
of the technical one.
There was a problem hiding this comment.
The taste-engine slide was intentionally removed per the user's explicit request: "Slide 12 does not need to be the taste engine, does not need to be the weekend models." This is not a regression — it was a deliberate removal.
There was a problem hiding this comment.
Skipped: comment is from another GitHub bot.
There was a problem hiding this comment.
The transition slide placement was explicitly requested by the user: "After slide six [Market], I would add a slide saying 'Now look at the technical product.'" The user wanted it here as a narrative bridge into the second half of the deck (business model, competition, and technical architecture). Keeping it as-is per the user's direction.
There was a problem hiding this comment.
Skipped: comment is from another GitHub bot.
1 participant
Summary
Overhauls the pitch deck (
deck/index.html) with visual slides, generated diagrams, and structural refinements based on iterative feedback.Structural changes:
tam-sam-som.png)New generated image assets (in
deck/slides/):problem-breakdown.png— 3 pain points visualsolution-breakdown.png— 3 solutions visualcompetitor-flows.png— step-by-step flow comparison (Elfster, Gifts.com, MyRegistry vs Giftmaxxing)real-competitor-comparison.png— side-by-side screenshot comparisonpipeline-diagram.png— 5-stage taste pipelinearchitecture.png— frontend/intelligence/infrastructure layerstam-sam-som.png— TAM $1T+ / SAM $100B+ / SOM concentric circlesFinal deck order (12 slides):
Link to Devin session: https://calhacks-promptetheus.devinenterprise.com/sessions/1833bca410e744b497d4e8afa05a93d9
Requested by: @Tar-ive
Summary by CodeRabbit