Skip to content
This repository has been archived by the owner on Aug 9, 2024. It is now read-only.

Coding challenge - Edgardo Zuniga #30

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

edzuniga
Copy link

This pull request is my approach to the Superformula flutter_test. Tried to focus on code organization, state management, and adherence to best practices.

The key highlights of this PR:

  1. Riverpod Integration for State Management: simplicity and efficiency approach. I introduced centralized state management, ensuring a more maintainable and scalable codebase.
  2. App Structure: structured in logical folders based on functionalities.
  3. State Management: tried to eliminate redundancy.
  4. Widget and Function Extraction: to minimize code duplication and improve maintainability.
  5. Packages used: collection, flutter_dotenv, gap, google_fonts, flutter_riverpod, and flutter_lorem.
  6. Adherence to Best Practices: I've tried to adhere to best practices for naming conventions, commenting, and overall code quality.

- Created RestaurantsNotifier and FavoritesNotifier for managing state of restaurant.
- Updated UI components to reflect state changes dynamically.
- This commit introduces Riverpod as the state management solution, enhancing the application's ability to handle state more efficiently and making the UI more responsive.
- Adjusted various UI elements for better alignment and consistency.
- Refactored components to improve code readability and maintainability.
- Fixed minor bugs affecting the responsiveness and display of UI elements.
This commit introduces unit tests for state management functionalities. Adiitionally, it includes a widget test for the BuildRestaurantItem() widget. Minor general fixes are also included.
- Minor UI improvements.
- adding comments to improve code understandability for future reference and collaboration.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant