Skip to content

Food Service - #3

Open
SarthakPaandey wants to merge 5 commits into
Scaler-Innovation-Labs:mainfrom
SarthakPaandey:main
Open

Food Service#3
SarthakPaandey wants to merge 5 commits into
Scaler-Innovation-Labs:mainfrom
SarthakPaandey:main

Conversation

@SarthakPaandey

Copy link
Copy Markdown

Implemented the food service.

@SarthakPaandey
SarthakPaandey requested a review from Copilot May 13, 2025 16:33

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

This PR implements the food service functionality by creating new API endpoints and dashboard components.

  • Adds API functions for vendor, vendor plan, and vendor plan selection management
  • Implements dashboard components for quick stats, mess menu, and mess feedback
  • Creates a new food services page integrating these components

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/lib/api/food-services.ts New API endpoints for food service operations
src/components/dashboard/food-services/QuickStats.tsx Dashboard component displaying vendor and plan stats
src/components/dashboard/food-services/MessMenu.tsx Component for displaying the monthly mess menu image
src/components/dashboard/food-services/MessFeedback.tsx Component enabling users to submit feedback
src/app/(dashboard)/food-services/page.tsx Main page integrating food service components

const { theme } = useTheme();
const isDark = theme === "dark";

const [selectedType, setSelectedType] = useState<string>(data.mealTypes[0] || '');

Copilot AI May 13, 2025

Copy link

Choose a reason for hiding this comment

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

The 'selectedType' state is defined but not used in the component. Either remove it or integrate it into the UI logic.

Copilot uses AI. Check for mistakes.
</div>
<div className="flex flex-wrap gap-2">
{data.mealTypes.map(type => {
const isSelected = type === 'Veg';

Copilot AI May 13, 2025

Copy link

Choose a reason for hiding this comment

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

The selection check for meal types is hard-coded to 'Veg'; consider using the 'selectedType' state to dynamically determine the selected meal type.

Suggested change
const isSelected = type === 'Veg';
const isSelected = type === selectedType;

Copilot uses AI. Check for mistakes.
@RickDeb2004

Copy link
Copy Markdown
Collaborator

image
here's the color of the div is not matching with figma file , the blue is more darker in the figma file. (check that exact hex code from figma)

@RickDeb2004

Copy link
Copy Markdown
Collaborator

@SarthakPaandey why Quick Stat component is not there ?Please check

Also the border radius of the divs (Mess Feed back ) is more than now , add box shadow there .

Please thoroughly check the figma

@RickDeb2004
RickDeb2004 changed the base branch from foodservice to main May 14, 2025 03:13
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.

3 participants