Skip to content

feat(SCRUM-98): SCRUM-98-location-page#17

Merged
mariam-mohamed00 merged 10 commits into
feature/SCRUM-80-home-orderfrom
feature/SCRUM-98-maps
Mar 16, 2026
Merged

feat(SCRUM-98): SCRUM-98-location-page#17
mariam-mohamed00 merged 10 commits into
feature/SCRUM-80-home-orderfrom
feature/SCRUM-98-maps

Conversation

@mariam-mohamed00
Copy link
Copy Markdown
Collaborator

No description provided.

@mariam-mohamed00 mariam-mohamed00 changed the title feat(SCRUM-98): SCRUM-98-location_page feat(SCRUM-98): SCRUM-98-location-page Mar 10, 2026
@elevate-flutter-team
Copy link
Copy Markdown
Collaborator

🚧 Automated Code Review

Score: 0/100

📋 Summary

  • ⚠️ Large PR: 46 files (Keep it small).
  • ❌ 13 functions exceed 20 lines.
  • ❌ 5 print statements found.
  • ⚠️ 7 hardcoded colors.
  • ❌ 2 naming convention errors.

🔍 Detected Issues

File Category Issue
...di.config.dart Naming File must be snake_case
di.config.dart...
...order_details_remote_datasource_impl.dart Maintainability Function 'getLatLngFromAddress' exceeds 20 lines
getLatLngFromAddress()...
...order_details_remote_datasource_impl.dart Clean Code Remove 'print' statements
print("<<<<<<<< Geocode response: $data"...
...order_details_cubit.dart Maintainability Function 'getOrderDetails' exceeds 20 lines
getOrderDetails()...
...drivers_orders_details_page.dart Maintainability Function 'build' exceeds 20 lines
build()...
...location_page.dart Maintainability Function 'build' exceeds 20 lines
build()...
...location_page.dart Naming Class '_LocationPageState' must be PascalCase
class _LocationPageState extends State<L...
...location_page.dart Clean Code Remove 'print' statements
print(...
...location_page.dart Design Use Theme.of(context) or AppColors
color: AppColors.pink,...
...location_page.dart Clean Code Remove 'print' statements
print(...
...location_page.dart Clean Code Remove 'print' statements
print(...
...location_page.dart Clean Code Remove 'print' statements
print(...
...location_page.dart Design Use Theme.of(context) or AppColors
backgroundColor: AppColors.pink,...
...location_page.dart Design Use Theme.of(context) or AppColors
color: AppColors.white,...
...address_card.dart Maintainability Function 'build' exceeds 20 lines
build()...

(...and 12 more issues)

Review generated for Mariam Mohamed

Comment thread lib/app/core/utils/app_launcher.dart Outdated
Comment thread lib/features/driver_orders_details/domain/usecases/location_usecase.dart Outdated
Comment thread lib/features/driver_orders_details/presentation/pages/location_page.dart Outdated
Comment thread lib/features/driver_orders_details/presentation/pages/location_page.dart Outdated
Comment thread lib/features/driver_orders_details/presentation/pages/location_page.dart Outdated
@mariam-mohamed00 mariam-mohamed00 merged commit a49f22d into feature/SCRUM-80-home-order Mar 16, 2026
3 checks passed
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