Feat/userstory5/styles #92
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…tionalities and styling for an alert notification feature (#80) * Feat/userstory4/styles (#74) * chore: git pull main * Add Supabase dependency and resolve TypeScript compilation issues * Implement comprehensive alert system with UI components and utilities - Add alertUtils.ts with 6 core utility functions for alert management - Create AlertBadge component with severity-based styling and variants - Add ThresholdSlider with interactive threshold setting and sensitivity indicators - Implement AlertIcon with SVG icons for different alert types - Build DismissButton with confirmation dialogs and loading states - Create NotificationIndicator with badges, dots, and positioning system - Add comprehensive CSS styling with accessibility and responsive design - Update utilities index to export all alert functionality Features: - TypeScript interfaces for AlertSeverity, AlertType, PriorityLevel, AttendanceAlert - Accessibility support with ARIA labels and keyboard navigation - Responsive design with mobile-friendly layouts - High contrast and reduced motion support - Print styles optimization - CSS custom properties for easy theming - Comprehensive component variants and presets * Delete package.json --------- Co-authored-by: Your Name <[email protected]> Co-authored-by: bscott519 <[email protected]> * feat: adds alert configuration modals for user story 4 (#78) * feat: add useAlertModal hook * feat: add ThresholdSettings Modal * feat: add alertDropdown * feat: add confirmation dialog for dismissing alerts * feat: update individual css files for alertdetailsmodal, alertworkflow integration, and notification settings * feat: update files to remove redundancies * feat: remove duplicates from useAlertModals file * feat: remove redundant css files * feat: remove parent notification settings * feat: update useAlertModal hook to remove notificationmodal states * User story 4 data validation (#76) * feat: add simple alert and threshold type definitions - Add SimpleAlertData interface for basic alert display - Add SimpleThresholdFormData for setting absence/lateness thresholds - Add SimpleAlertCalculationResult for threshold monitoring - Add alert filtering, sorting, and summary statistics interfaces - Add default threshold values and validation constants - Extend AttendanceAlert and AlertThreshold domains with supporting interfaces - Implement consistent ValidationResult pattern for all validation functions * feat: implement basic threshold and alert validation - Add alertValidation.ts with functions for alert data validation - Add thresholdValidation.ts with threshold form validation logic - Implement validateAlertData() for alert structure validation - Implement validateThresholdForm() with comprehensive field checking - Add sanitizeThresholdInput() for cleaning user input - Add validateTimeframe() for date range validation - Include cross-field validation for logical threshold relationships - Use consistent ValidationResult pattern throughout * feat: add simple alert calculation utilities - Add alertCalculations.ts with core attendance counting logic - Implement calculateStudentAlerts() for threshold monitoring - Add countAbsencesInPeriod() and countLatenessInPeriod() for rolling window calculations - Add countAbsencesCumulative() and countLatenessCumulative() for total counts - Implement calculateDaysOverThreshold() for duration tracking - Add batch calculation support for multiple students - Include attendance trend analysis and threshold proximity checking - Add date range filtering and validation utilities * test: add comprehensive validation and calculation tests - Add alertValidation.test.ts with validation function tests - Add thresholdValidation.test.ts with threshold form tests - Add alertCalculations.test.ts with calculation logic tests - Ensure 80% code coverage for alert system utilities - Test edge cases and error conditions for robust validation - Support core attendance threshold monitoring requirements * docs: add comprehensive alert system documentation - Add docs/alert-system.md with complete usage guide - Include JSDoc comments for key validation functions - Document integration patterns and error handling - Provide code examples for common use cases - Cover testing strategy and performance considerations - Support core attendance threshold monitoring requirements * fix: remove AlertSeverity reference for simplified alert system - Change getSeverityLevel() return type from AlertSeverity to string - Maintain compatibility with simplified alert architecture - Ensures clean TypeScript compilation without build errors * fix: resolve ESLint issues in User Story 4 files - Remove unused imports of AlertType and AlertPeriod - Replace 'any' types with 'unknown' for better type safety - Ensure clean lint compliance for alert validation system - Maintain code quality standards for GitHub CI/CD pipeline * chore: add ESLint configuration file - Auto-generated ESLint config from npm run lint - Ensures consistent code style across the project * feat: implemented alerts page (#75) * created files for user story 3 requirements * implementing interactive calendar components * finished created interactive calendar and redesigned dashboard * implemented alerts dashboard * edited css for the intervention form --------- Co-authored-by: Benjamin Scott <[email protected]> Co-authored-by: Jared Edge <[email protected]> * Story 4 dev (#79) * feat(validation): implement User Story 3 schedule data validation foundation (#66) • Add comprehensive Zod validation schemas for schedule and day-off operations • Implement date validation utilities with business rule enforcement • Create reason validation with smart suggestion system • Build centralized validation orchestration with conflict detection • Establish type-safe interfaces for team integration • Achieve 81% test coverage with 64 comprehensive unit tests * Feat: adds schedule API endpoints including days off and excused days * Feat: adds Delete endpoint and API validation * Fix: Ensure data directory and schedule.json file exist for Vercel deployment * Feat: adds domain models and database schema for the alert system * Feat: adds Alerting logic and API endpoints for alerts * Chore: fix API endpoints and failed vercel build * Implement RAG for alert explanations (User Story 4) * Feat: adds combines modals into one page to improve UX * Feat: fixes reports page UI * Feat: adds environment Setup & dependencies for openAI API * Feat: adds fix vercel build issues with dotenv and OpenAI dependencies * Feat: adds system prompts, LLM to support context, updayed RAG to use prompts, and added tests * Feat: adds OpenAI client configuration, Query Processing and Sanitization, Response Handling and Validation, Additional Improvements * Feat: adds Response Formatting & Validation Implementation. Output Structure Standardization, Error Handling , Configuration Updates, Integration Improvements * Feat: adapter between domains, Fixed type compatibility, Fixed date formatting and adjusted filter handling * Feat: added test files for rag and llm. * Feat: adds corrected endpoints, proper logic and formating for LLM response, DATA formatiing for response fix * fix: response not appearing after user prompt --------- Co-authored-by: lquinoa252 <[email protected]> * chore: resolves merge confilct * chore: removes strict lint rules for deployment * feat: adds removes build blocks * chore: fixes duplicate methods * feat: removes mention of supabase and fixes student mapping --------- Co-authored-by: tboyle252-sudo <[email protected]> Co-authored-by: Your Name <[email protected]> Co-authored-by: bscott519 <[email protected]> Co-authored-by: imo-252 <[email protected]> Co-authored-by: lquinoa252 <[email protected]> Co-authored-by: Benjamin Scott <[email protected]>
* feat(types): add core student interfaces and form types - src/types/students.ts: StudentProfile, Student, StudentFilters interfaces - src/types/studentForm.ts: StudentFormData, ValidationResult interfaces - Matches existing API patterns and avoids domain conflicts - Supports grade as optional string per API expectations - Includes bulk operations and sorting for team integration * feat(constants): add student validation constants and rules - src/constants/studentConstants.ts: validation patterns, limits, error messages * feat(validation): implement student validation utilities - src/utils/studentValidation.ts: validateStudentId, validateStudentName, isValidGrade, generateNextStudentId functions * feat(validation): add form validation helpers - src/utils/formValidation.ts: validateStudentForm, validateField, form state management utilities * feat(utils): add student data transformation helpers - src/utils/studentHelpers.ts: sanitizeStudentInput, data transformation, formatting utilities * test: add validation tests and finalize student validation suite - tests/unit/studentValidation.test.ts: comprehensive validation function tests - tests/unit/formValidation.test.ts: form validation and state management tests - src/utils/formValidation.ts: fix validateField function for accurate field-specific validation
… application (#88) * Feat/userstory4/styles (#74) * chore: git pull main * Add Supabase dependency and resolve TypeScript compilation issues * Implement comprehensive alert system with UI components and utilities - Add alertUtils.ts with 6 core utility functions for alert management - Create AlertBadge component with severity-based styling and variants - Add ThresholdSlider with interactive threshold setting and sensitivity indicators - Implement AlertIcon with SVG icons for different alert types - Build DismissButton with confirmation dialogs and loading states - Create NotificationIndicator with badges, dots, and positioning system - Add comprehensive CSS styling with accessibility and responsive design - Update utilities index to export all alert functionality Features: - TypeScript interfaces for AlertSeverity, AlertType, PriorityLevel, AttendanceAlert - Accessibility support with ARIA labels and keyboard navigation - Responsive design with mobile-friendly layouts - High contrast and reduced motion support - Print styles optimization - CSS custom properties for easy theming - Comprehensive component variants and presets * Delete package.json --------- Co-authored-by: Your Name <[email protected]> Co-authored-by: bscott519 <[email protected]> * feat: adds alert configuration modals for user story 4 (#78) * feat: add useAlertModal hook * feat: add ThresholdSettings Modal * feat: add alertDropdown * feat: add confirmation dialog for dismissing alerts * feat: update individual css files for alertdetailsmodal, alertworkflow integration, and notification settings * feat: update files to remove redundancies * feat: remove duplicates from useAlertModals file * feat: remove redundant css files * feat: remove parent notification settings * feat: update useAlertModal hook to remove notificationmodal states * User story 4 data validation (#76) * feat: add simple alert and threshold type definitions - Add SimpleAlertData interface for basic alert display - Add SimpleThresholdFormData for setting absence/lateness thresholds - Add SimpleAlertCalculationResult for threshold monitoring - Add alert filtering, sorting, and summary statistics interfaces - Add default threshold values and validation constants - Extend AttendanceAlert and AlertThreshold domains with supporting interfaces - Implement consistent ValidationResult pattern for all validation functions * feat: implement basic threshold and alert validation - Add alertValidation.ts with functions for alert data validation - Add thresholdValidation.ts with threshold form validation logic - Implement validateAlertData() for alert structure validation - Implement validateThresholdForm() with comprehensive field checking - Add sanitizeThresholdInput() for cleaning user input - Add validateTimeframe() for date range validation - Include cross-field validation for logical threshold relationships - Use consistent ValidationResult pattern throughout * feat: add simple alert calculation utilities - Add alertCalculations.ts with core attendance counting logic - Implement calculateStudentAlerts() for threshold monitoring - Add countAbsencesInPeriod() and countLatenessInPeriod() for rolling window calculations - Add countAbsencesCumulative() and countLatenessCumulative() for total counts - Implement calculateDaysOverThreshold() for duration tracking - Add batch calculation support for multiple students - Include attendance trend analysis and threshold proximity checking - Add date range filtering and validation utilities * test: add comprehensive validation and calculation tests - Add alertValidation.test.ts with validation function tests - Add thresholdValidation.test.ts with threshold form tests - Add alertCalculations.test.ts with calculation logic tests - Ensure 80% code coverage for alert system utilities - Test edge cases and error conditions for robust validation - Support core attendance threshold monitoring requirements * docs: add comprehensive alert system documentation - Add docs/alert-system.md with complete usage guide - Include JSDoc comments for key validation functions - Document integration patterns and error handling - Provide code examples for common use cases - Cover testing strategy and performance considerations - Support core attendance threshold monitoring requirements * fix: remove AlertSeverity reference for simplified alert system - Change getSeverityLevel() return type from AlertSeverity to string - Maintain compatibility with simplified alert architecture - Ensures clean TypeScript compilation without build errors * fix: resolve ESLint issues in User Story 4 files - Remove unused imports of AlertType and AlertPeriod - Replace 'any' types with 'unknown' for better type safety - Ensure clean lint compliance for alert validation system - Maintain code quality standards for GitHub CI/CD pipeline * chore: add ESLint configuration file - Auto-generated ESLint config from npm run lint - Ensures consistent code style across the project * feat: implemented alerts page (#75) * created files for user story 3 requirements * implementing interactive calendar components * finished created interactive calendar and redesigned dashboard * implemented alerts dashboard * edited css for the intervention form --------- Co-authored-by: Benjamin Scott <[email protected]> Co-authored-by: Jared Edge <[email protected]> * Story 4 dev (#79) * feat(validation): implement User Story 3 schedule data validation foundation (#66) • Add comprehensive Zod validation schemas for schedule and day-off operations • Implement date validation utilities with business rule enforcement • Create reason validation with smart suggestion system • Build centralized validation orchestration with conflict detection • Establish type-safe interfaces for team integration • Achieve 81% test coverage with 64 comprehensive unit tests * Feat: adds schedule API endpoints including days off and excused days * Feat: adds Delete endpoint and API validation * Fix: Ensure data directory and schedule.json file exist for Vercel deployment * Feat: adds domain models and database schema for the alert system * Feat: adds Alerting logic and API endpoints for alerts * Chore: fix API endpoints and failed vercel build * Implement RAG for alert explanations (User Story 4) * Feat: adds combines modals into one page to improve UX * Feat: fixes reports page UI * Feat: adds environment Setup & dependencies for openAI API * Feat: adds fix vercel build issues with dotenv and OpenAI dependencies * Feat: adds system prompts, LLM to support context, updayed RAG to use prompts, and added tests * Feat: adds OpenAI client configuration, Query Processing and Sanitization, Response Handling and Validation, Additional Improvements * Feat: adds Response Formatting & Validation Implementation. Output Structure Standardization, Error Handling , Configuration Updates, Integration Improvements * Feat: adapter between domains, Fixed type compatibility, Fixed date formatting and adjusted filter handling * Feat: added test files for rag and llm. * Feat: adds corrected endpoints, proper logic and formating for LLM response, DATA formatiing for response fix * fix: response not appearing after user prompt --------- Co-authored-by: lquinoa252 <[email protected]> * chore: resolves merge confilct * chore: removes strict lint rules for deployment * feat: adds removes build blocks * chore: fixes duplicate methods * feat: removes mention of supabase and fixes student mapping * feat: implementing student management functionality (#84) * created files for student management page * implementing student management page components * created settings page and edited students page and dashboard * fixed sidebar not appearing when on settings page and adjusted background * Delete app/globals.css --------- * feat: implement comprehensive student modal system with 6 key functions (#83) ✨ New Implementation: • useStudentModals hook with centralized state management and workflow control • Complete modal system: Add, Edit, Delete, Form, and BulkActions components • Six core functions: useStudentModals(), handleStudentFormSubmit(), handleDeleteConfirmation(), showDependencyWarning(), handleFormValidation(), handleModalClose() 🔧 Key Features: • Real-time form validation with error display and user feedback • Unsaved changes protection with confirmation dialogs • Dependency checking and warning system for data relationships • Bulk operations: delete, export, archive, grade updates, notifications • Type-safe interfaces with comprehensive TypeScript integration • Responsive design with accessibility features and proper UX 🎯 Modal Components: • AddStudentModal: Student creation with validation and success handling • EditStudentModal: Student updates with pre-populated data and change tracking • DeleteStudentModal: Safe deletion with dependency warnings and confirmation • StudentFormModal: Universal create/edit form with mode-based initialization • BulkActionsModal: Multi-student operations with safety checks and progress tracking 📋 Technical Implementation: • Centralized useStudentModals hook for consistent state management across all modals • Form validation with real-time error checking and visual feedback • Modal workflow management with proper open/close state handling • Dependency resolution system for data integrity protection • Loading states and error handling for improved user experience • TailwindCSS styling with responsive grid layouts and accessibility compliance 🚀 Integration Ready: All modals integrate seamlessly with existing student management system and are ready for production use with complete error handling and type safety. * Feat: adds API routing logic for student management to create, delete, and move a student throughout the system (#81) * Feat: adds methods to configure a student in the application * Feat: adds Business llogic layer, enhancements to the student repo, and domain validation * Feat: adds API routes for individual and bulk student operations await service methods * Feat: adds unit test for student management * Feat: adds proper http method for API routing * User story 5 data validation (#87) * All team members added required files for User Story 4, Includes functionalities and styling for an alert notification feature (#80) * Feat/userstory4/styles (#74) * chore: git pull main * Add Supabase dependency and resolve TypeScript compilation issues * Implement comprehensive alert system with UI components and utilities - Add alertUtils.ts with 6 core utility functions for alert management - Create AlertBadge component with severity-based styling and variants - Add ThresholdSlider with interactive threshold setting and sensitivity indicators - Implement AlertIcon with SVG icons for different alert types - Build DismissButton with confirmation dialogs and loading states - Create NotificationIndicator with badges, dots, and positioning system - Add comprehensive CSS styling with accessibility and responsive design - Update utilities index to export all alert functionality Features: - TypeScript interfaces for AlertSeverity, AlertType, PriorityLevel, AttendanceAlert - Accessibility support with ARIA labels and keyboard navigation - Responsive design with mobile-friendly layouts - High contrast and reduced motion support - Print styles optimization - CSS custom properties for easy theming - Comprehensive component variants and presets * Delete package.json --------- Co-authored-by: Your Name <[email protected]> Co-authored-by: bscott519 <[email protected]> * feat: adds alert configuration modals for user story 4 (#78) * feat: add useAlertModal hook * feat: add ThresholdSettings Modal * feat: add alertDropdown * feat: add confirmation dialog for dismissing alerts * feat: update individual css files for alertdetailsmodal, alertworkflow integration, and notification settings * feat: update files to remove redundancies * feat: remove duplicates from useAlertModals file * feat: remove redundant css files * feat: remove parent notification settings * feat: update useAlertModal hook to remove notificationmodal states * User story 4 data validation (#76) * feat: add simple alert and threshold type definitions - Add SimpleAlertData interface for basic alert display - Add SimpleThresholdFormData for setting absence/lateness thresholds - Add SimpleAlertCalculationResult for threshold monitoring - Add alert filtering, sorting, and summary statistics interfaces - Add default threshold values and validation constants - Extend AttendanceAlert and AlertThreshold domains with supporting interfaces - Implement consistent ValidationResult pattern for all validation functions * feat: implement basic threshold and alert validation - Add alertValidation.ts with functions for alert data validation - Add thresholdValidation.ts with threshold form validation logic - Implement validateAlertData() for alert structure validation - Implement validateThresholdForm() with comprehensive field checking - Add sanitizeThresholdInput() for cleaning user input - Add validateTimeframe() for date range validation - Include cross-field validation for logical threshold relationships - Use consistent ValidationResult pattern throughout * feat: add simple alert calculation utilities - Add alertCalculations.ts with core attendance counting logic - Implement calculateStudentAlerts() for threshold monitoring - Add countAbsencesInPeriod() and countLatenessInPeriod() for rolling window calculations - Add countAbsencesCumulative() and countLatenessCumulative() for total counts - Implement calculateDaysOverThreshold() for duration tracking - Add batch calculation support for multiple students - Include attendance trend analysis and threshold proximity checking - Add date range filtering and validation utilities * test: add comprehensive validation and calculation tests - Add alertValidation.test.ts with validation function tests - Add thresholdValidation.test.ts with threshold form tests - Add alertCalculations.test.ts with calculation logic tests - Ensure 80% code coverage for alert system utilities - Test edge cases and error conditions for robust validation - Support core attendance threshold monitoring requirements * docs: add comprehensive alert system documentation - Add docs/alert-system.md with complete usage guide - Include JSDoc comments for key validation functions - Document integration patterns and error handling - Provide code examples for common use cases - Cover testing strategy and performance considerations - Support core attendance threshold monitoring requirements * fix: remove AlertSeverity reference for simplified alert system - Change getSeverityLevel() return type from AlertSeverity to string - Maintain compatibility with simplified alert architecture - Ensures clean TypeScript compilation without build errors * fix: resolve ESLint issues in User Story 4 files - Remove unused imports of AlertType and AlertPeriod - Replace 'any' types with 'unknown' for better type safety - Ensure clean lint compliance for alert validation system - Maintain code quality standards for GitHub CI/CD pipeline * chore: add ESLint configuration file - Auto-generated ESLint config from npm run lint - Ensures consistent code style across the project * feat: implemented alerts page (#75) * created files for user story 3 requirements * implementing interactive calendar components * finished created interactive calendar and redesigned dashboard * implemented alerts dashboard * edited css for the intervention form --------- Co-authored-by: Benjamin Scott <[email protected]> Co-authored-by: Jared Edge <[email protected]> * Story 4 dev (#79) * feat(validation): implement User Story 3 schedule data validation foundation (#66) • Add comprehensive Zod validation schemas for schedule and day-off operations • Implement date validation utilities with business rule enforcement • Create reason validation with smart suggestion system • Build centralized validation orchestration with conflict detection • Establish type-safe interfaces for team integration • Achieve 81% test coverage with 64 comprehensive unit tests * Feat: adds schedule API endpoints including days off and excused days * Feat: adds Delete endpoint and API validation * Fix: Ensure data directory and schedule.json file exist for Vercel deployment * Feat: adds domain models and database schema for the alert system * Feat: adds Alerting logic and API endpoints for alerts * Chore: fix API endpoints and failed vercel build * Implement RAG for alert explanations (User Story 4) * Feat: adds combines modals into one page to improve UX * Feat: fixes reports page UI * Feat: adds environment Setup & dependencies for openAI API * Feat: adds fix vercel build issues with dotenv and OpenAI dependencies * Feat: adds system prompts, LLM to support context, updayed RAG to use prompts, and added tests * Feat: adds OpenAI client configuration, Query Processing and Sanitization, Response Handling and Validation, Additional Improvements * Feat: adds Response Formatting & Validation Implementation. Output Structure Standardization, Error Handling , Configuration Updates, Integration Improvements * Feat: adapter between domains, Fixed type compatibility, Fixed date formatting and adjusted filter handling * Feat: added test files for rag and llm. * Feat: adds corrected endpoints, proper logic and formating for LLM response, DATA formatiing for response fix * fix: response not appearing after user prompt --------- Co-authored-by: lquinoa252 <[email protected]> * chore: resolves merge confilct * chore: removes strict lint rules for deployment * feat: adds removes build blocks * chore: fixes duplicate methods * feat: removes mention of supabase and fixes student mapping --------- Co-authored-by: tboyle252-sudo <[email protected]> Co-authored-by: Your Name <[email protected]> Co-authored-by: bscott519 <[email protected]> Co-authored-by: imo-252 <[email protected]> Co-authored-by: lquinoa252 <[email protected]> Co-authored-by: Benjamin Scott <[email protected]> * feat(types): add core student interfaces and form types - src/types/students.ts: StudentProfile, Student, StudentFilters interfaces - src/types/studentForm.ts: StudentFormData, ValidationResult interfaces - Matches existing API patterns and avoids domain conflicts - Supports grade as optional string per API expectations - Includes bulk operations and sorting for team integration * feat(constants): add student validation constants and rules - src/constants/studentConstants.ts: validation patterns, limits, error messages * feat(validation): implement student validation utilities - src/utils/studentValidation.ts: validateStudentId, validateStudentName, isValidGrade, generateNextStudentId functions * feat(validation): add form validation helpers - src/utils/formValidation.ts: validateStudentForm, validateField, form state management utilities * feat(utils): add student data transformation helpers - src/utils/studentHelpers.ts: sanitizeStudentInput, data transformation, formatting utilities * test: add validation tests and finalize student validation suite - tests/unit/studentValidation.test.ts: comprehensive validation function tests - tests/unit/formValidation.test.ts: form validation and state management tests - src/utils/formValidation.ts: fix validateField function for accurate field-specific validation --------- Co-authored-by: Jared Edge <[email protected]> Co-authored-by: tboyle252-sudo <[email protected]> Co-authored-by: Your Name <[email protected]> Co-authored-by: bscott519 <[email protected]> Co-authored-by: imo-252 <[email protected]> Co-authored-by: Benjamin Scott <[email protected]> * Chore: removes global.css import --------- Co-authored-by: tboyle252-sudo <[email protected]> Co-authored-by: Your Name <[email protected]> Co-authored-by: bscott519 <[email protected]> Co-authored-by: imo-252 <[email protected]> Co-authored-by: lquinoa252 <[email protected]> Co-authored-by: Benjamin Scott <[email protected]>
- Integrated key utility functions into student management components: - StudentAvatar: formatStudentName, generateStudentInitials, getStudentAvatarColor - RequiredFieldLabel: isRequiredField auto-detection - ValidationMessage: field name formatting - StudentBadge: formatGradeDisplay, formatStudentId auto-formatting - ConfirmButton: dynamic student name formatting - Fixed ESLint errors by adding proper displayName to sub-components - Updated StudentAvatar to use Next.js Image component - Fixed classes index exports to prevent build errors - All components now pass TypeScript and linting checks - Build successfully completes with no errors
- Resolved merge conflict in components/utilities/index.ts - Kept student utility function exports for component integration - Maintained compatibility with latest remote changes
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added files according to the overview.