search

woltapp / wolt_modal_sheet

This package provides a responsive modal with multiple pages, motion animation for page transitions, and scrollable content within each page.
MIT License
511 stars 64 forks source link

[Coffee Maker Demo app] Added dependency injection explanation in code diagram to main.dart #316

Closed ulusoyca closed 2 months ago

ulusoyca commented 2 months ago

Description

Added the following diagram to explain the wolt_di framework in main.dart.

/*
+----------------------------------+
|  DependencyContainerManager      |       +----------------------------------+
|  +-----------------------------+ |       |                                  |
|  | App Level DI Container      | |       |          MaterialApp             |
|  | +-------------------------+ | |       |  +-----------------------------+ |
|  | | AuthService             | | |       |  |    DependencyInjector       | |
|  | +-------------------------+ | |       |  |          Widget             | |
|  | | AuthRepository          | | |       |  |  +-----------------------+  | |
|  | +-------------------------+ | |       |  |  | FeatureLevelDependency|  | |
|  | | AuthRemoteDataSource    | | |       |  |  |        Container      |  | |
|  | +-------------------------+ | |       |  |  | +-------------------+ |  | |
|  | | RouterViewModel         | | |       |  |  | |      Feature      | |  | |
|  | +-------------------------+ | |       |  |  | |    Screen Widget  | |  | |
|  +-----------------------------+ |       |  |  | |                   | |  | |
|                                  |       |  |  | +-------------------+ |  | |
|  +-----------------------------+ |       |  |  |                       |  | |
|  | OrdersDependencyContainer   | |       |  |  +-----------------------+  | |
|  | +-------------------------+ | |       |  +-----------------------------+ |
|  | | OrderService            | | |       |                                  |
|  | +-------------------------+ | |       +----------------------------------+
|  | | OrdersRepository        | | |
|  | +-------------------------+ | |
|  | | OrdersRemoteDataSource  | | |
|  | +-------------------------+ | |
|  | | OrderScreenVM (Factory) | | |
|  | +-------------------------+ | |
|  +-----------------------------+ |
|                                  |
|  +-----------------------------+ |
|  | AddWaterDependencyContainer | |
|  | +-------------------------+ | |
|  | |AddWaterScreenVM(Factory)| | |
|  | +-------------------------+ | |
|  | | AddWaterService         | | |
|  | +-------------------------+ | |
|  +-----------------------------+ |
+----------------------------------+
*/
github-actions[bot] commented 2 months ago

It appears that this PR does not include any tests. It is recommended to add tests, especially for critical changes, to ensure code quality and prevent regressions. However, if this PR is only updating samples or documentation, feel free to skip adding tests and disregard this comment.