Conversation
- Created demo_seed_data.dart for mock services and appointments. - Implemented validators.dart for form validation (required fields, email, password). - Added AdminBottomNav and ClientBottomNav widgets for navigation. - Introduced ErrorView and LoadingView widgets for better user feedback. - Added SectionHeader widget for consistent section titles. - Updated generated_plugin_registrant.cc and generated_plugins.cmake for secure storage support on Linux and Windows. - Updated pubspec.yaml to include dio, flutter_secure_storage, and intl dependencies. - Added mvp_flutter_starter_guide.md for project structure and implementation guidance.
…ement - Created package.json for mock API with json-server dependency. - Implemented server.js to handle user authentication and appointment scheduling. - Added endpoints for user login and fetching user appointments. - Included middleware for request parsing and response formatting. test: update widget test to verify app structure - Refactored widget test to check for the main app widget. - Removed counter-related tests and focused on app initialization.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (51)
📝 WalkthroughWalkthroughThis pull request introduces a complete Flutter MVP application for a service booking system. It adds project structure with core infrastructure (networking, routing, storage, theming), feature modules for authentication/services/bookings with Riverpod state management, multiple UI pages for client and admin roles, shared utilities and widgets, a mock backend with json-server, and comprehensive documentation guides. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~75 minutes
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This pull request introduces foundational improvements and best practices for a beginner-friendly Flutter project, including complete onboarding documentation, a robust API and network layer, consistent routing, and a shared design system. The changes are grouped below by theme.
Documentation and Onboarding
flutter-development-guide.mdcovering project scope, recommended dependencies, folder structure, backend contract checklist, build plan, and common mistakes, tailored for beginners working on a class project.README.mdwith a clear folder structure, widget suggestions per page, project rules, and setup instructions for running with a mock backend.Core Infrastructure: Network and Configuration
Envclass inlib/core/config/env.dartto provide environment-aware backend base URLs, supporting both web and emulator scenarios.dioProviderinlib/core/network/dio_client.dart, including a secure storage-based token interceptor for authenticated requests.ApiServiceclass inlib/core/network/api_service.dartwith typed methods for authentication, services, and appointments, plus robust error handling and response mapping.Routing and Navigation
lib/core/router/route_names.dartfor both client and admin flows.appRouterProviderinlib/core/router/app_router.dart, handling authentication state, role-based redirects, and error fallback routes using GoRouter.Design System
AppColorsinlib/core/shared_widgets/app_colors.dart, defining a consistent color palette for primary, accent, background, text, and feedback states.Tooling
Summary by CodeRabbit
New Features
Chores