Skip to content

Conversation

@dxb2306
Copy link
Member

@dxb2306 dxb2306 commented Dec 19, 2025

Summary by Sourcery

Rework the homepage and navigation to present DMP Chef as an open-source Python pipeline with clearer context, partners, and funding, and to position the web app as a test interface for the pipeline rather than the primary product.

New Features:

  • Add a homepage hero with DMP Chef branding, updated overview copy, and links to the GitHub repository and test interface.
  • Introduce a Team section highlighting collaborating organizations with their logos, and a Funding section describing project sponsors.

Enhancements:

  • Refine homepage layout and styling, including a new logo display, updated background effects, and reorganized content sections.
  • Update navigation labels and CTAs to point to the GitHub repository and clarify that the app pages are for testing the DMP Chef pipeline.
  • Retire or comment out older feature and technology sections that no longer match the new positioning of the project.

@fairdataihub-bot
Copy link

Thank you for submitting this pull request! We appreciate your contribution to the project. Before we can merge it, we need to review the changes you've made to ensure they align with our code standards and meet the requirements of the project. We'll get back to you as soon as we can with feedback. Thanks again!

@vercel
Copy link

vercel bot commented Dec 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dmp-chef Ready Ready Preview, Comment Feb 11, 2026 2:47am
dmpchef-website Ready Ready Preview, Comment Feb 11, 2026 2:47am

Request Review

@sourcery-ai
Copy link

sourcery-ai bot commented Dec 19, 2025

Reviewer's Guide

Reworks the marketing and navigation aspects of the DMP Chef homepage and app to position it as an open-source pipeline with a testing interface, adds team and funding sections, updates navigation labels and GitHub entry points, and introduces logo assets used across the new layout.

Flow diagram for updated navigation and page structure

flowchart TD
  subgraph Layout_default_vue
    HEADER[Header_navigation_menu]
    FOOTER[Footer_links]
  end

  subgraph Homepage_index_vue
    HERO[UPageHero_with_View_GitHub_repo_and_Test_it_links]
    OVERVIEW[Overview_section_describing_open_source_pipeline]
    TEAM[Team_and_collaboration_section]
    FUNDING[Funding_section]
  end

  subgraph Testing_and_download_pages
    TEST_DMP[app_dmp_vue_Test_DMP_Chef]
    TEST_DMP1[app_dmp1_vue_Test_DMP_Chef]
    DOWNLOAD[app_download_vue_Test_DMP_Chef_download]
  end

  HEADER -->|GitHub_item| GITHUB_DEST[External_GitHub_repo]
  HEADER -->|Test_item| TEST_DMP

  HERO -->|View_GitHub_repo| GITHUB_DEST
  HERO -->|Test_it| TEST_DMP

  TEST_DMP --> TEST_DMP1
  TEST_DMP --> DOWNLOAD

  FOOTER --> COMMUNITY[Community_links_section]

  OVERVIEW --> TEAM
  TEAM --> FUNDING

  LOGO[Logo_assets_in_public_logos] --> HERO
  LOGO --> TEAM
Loading

File-Level Changes

Change Details Files
Retheme and expand the homepage hero and overview content to emphasize DMP Chef as an open-source Python pipeline and testing interface.
  • Updated homepage CTA links to point to the GitHub repo (opening in a new tab) and to the testing page instead of generic getting-started/learn-more actions
  • Adjusted the hero title/description from beta marketing copy to a neutral description of the DMP Chef pipeline
  • Reworked the overview section markup and styling, including background glow effects, constrained text width, and revised copy describing the pipeline, its purpose, and relationship to DMPTool
  • Replaced the old brief overview and removed the features list in favor of richer explanatory content about the pipeline and its goals
app/pages/index.vue
Add new informational sections for Team and Funding and hide lower-priority marketing sections.
  • Introduced a Team section with descriptive text and logo grid linking to FAIR Data Innovations Hub, California Digital Library, and DMPTool
  • Added a Funding section describing NSF and CZI support and partner organizations, with external links styled consistently
  • Commented out previous feature, technology, and secondary hero sections to simplify and focus the homepage content
app/pages/index.vue
Update navigation to expose GitHub and clarify that the DMP pages are for testing the pipeline.
  • Changed the main header navigation to include a GitHub link (opening in a new tab) and renamed the DMP app route item to Test
  • Commented out the header GitHub icon button and auth-related buttons in the default layout to simplify the header actions
  • Retitled the app DMP pages and download page headings from Draft DMP to Test DMP Chef and added explanatory text on the main DMP test page clarifying its purpose as a test harness for the pipeline
app/layouts/default.vue
app/pages/app/dmp.vue
app/pages/app/dmp1.vue
app/pages/app/download.vue
Add and wire up branding assets used on the homepage and team section.
  • Placed the DMP Chef logo above the homepage hero with centered layout and fixed sizing
  • Added SVG logo assets for FAIR Data Innovations Hub, California Digital Library, DMPTool, and DMP Chef (transparent and white variants) under the public/logos directory for use in the new sections
app/pages/index.vue
public/logos/dmp_tool_logo.svg
public/logos/fair_logo.svg
public/logos/logo-cdl.svg
public/logos/logo-dmpchef-transparent.svg
public/logos/logo-dmpchef-white.svg

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@fairdataihub-bot
Copy link

Thanks for making updates to your pull request. Our team will take a look and provide feedback as soon as possible. Please wait for any GitHub Actions to complete before editing your pull request. If you have any additional questions or concerns, feel free to let us know. Thank you for your contributions!

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've found 1 issue, and left some high level feedback:

  • The logo image on the homepage uses src="/../public/logos/logo-dmpchef-transparent.svg", which is inconsistent with the other assets in /public/logos and may not resolve correctly; consider updating it to a root-relative path like /logos/logo-dmpchef-transparent.svg.
  • There are several large sections now commented out in index.vue (features, extra cards, and a hero block); if these are no longer needed, it would be cleaner to remove them entirely or move them into a separate WIP branch/component to avoid long-term commented code in the main page.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The logo image on the homepage uses `src="/../public/logos/logo-dmpchef-transparent.svg"`, which is inconsistent with the other assets in `/public/logos` and may not resolve correctly; consider updating it to a root-relative path like `/logos/logo-dmpchef-transparent.svg`.
- There are several large sections now commented out in `index.vue` (features, extra cards, and a hero block); if these are no longer needed, it would be cleaner to remove them entirely or move them into a separate WIP branch/component to avoid long-term commented code in the main page.

## Individual Comments

### Comment 1
<location> `app/pages/index.vue:130-131` </location>
<code_context>
     <div class="absolute inset-x-0 top-0 h-40 bg-gradient-to-b from-blue-300 to-white dark:from-blue-800 dark:to-transparent -z-10"></div>
+    <div class="flex flex-col items-center">
+      <div class="flex justify-center mt-30">
+        <img 
+          src="/../public/logos/logo-dmpchef-transparent.svg" 
+          alt="DMP Chef Logo" 
+          class="h-30 w-auto" 
</code_context>

<issue_to_address>
**issue (bug_risk):** Logo asset path is likely incorrect for Nuxt public/ directory usage.

Other images here use `/logos/...`, which is the standard way to reference `public/` assets in Nuxt. `src="/../public/logos/..."` may fail in production or when served from a sub-path. Please update this to use the direct public path (e.g. `src="/logos/logo-dmpchef-transparent.svg"`) for consistency and reliability.
</issue_to_address>

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Comment on lines 130 to 131
<img
src="/../public/logos/logo-dmpchef-transparent.svg"
Copy link

Choose a reason for hiding this comment

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

issue (bug_risk): Logo asset path is likely incorrect for Nuxt public/ directory usage.

Other images here use /logos/..., which is the standard way to reference public/ assets in Nuxt. src="/../public/logos/..." may fail in production or when served from a sub-path. Please update this to use the direct public path (e.g. src="/logos/logo-dmpchef-transparent.svg") for consistency and reliability.

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.

2 participants