Skip to content

Comments

Feature/account setup#88

Open
Mostafa-alsaygh wants to merge 3 commits intodevfrom
feature/account-setup
Open

Feature/account setup#88
Mostafa-alsaygh wants to merge 3 commits intodevfrom
feature/account-setup

Conversation

@Mostafa-alsaygh
Copy link
Collaborator

make a page3 of setup account screen

Screen_recording_20260213_111219.mp4

- Add `categories` and `suggestions` to `AccountSetupState`.
- Implement `toggleCategory` logic in `AccountSetupCubit`.
- Create `Page2` for the account setup flow to allow users to select or add spending categories.
- Introduce `SelectedCategoryItem` widget and `icon_cancel_category.svg` asset for the category selection UI.
- Update `AccountSetupScreen` to support multi-page navigation and validation for the finish button.
- Add `categories` and `suggestions` to `AccountSetupState`.
- Implement `toggleCategory` logic in `AccountSetupCubit`.
- Create `Page2` for the account setup flow to allow users to select or add spending categories.
- Introduce `SelectedCategoryItem` widget and `icon_cancel_category.svg` asset for the category selection UI.
- Update `AccountSetupScreen` to support multi-page navigation and validation for the finish button.
# Conflicts:
#	lib/design_system/widgets/text_field.dart
),
const SizedBox(height: 24),
MTextField(
hint: 'Category name',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

localization pls

import '../../../design_system/theme/money_extension_context.dart';
import '../../../design_system/widgets/selected_category_item.dart';

class Page3 extends StatefulWidget {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think AccountSetupStepThree would be better and if tell what this step does would be much better
like if it's categories preferences AccountSetupPreferencesStep

@mohamedshemees
Copy link
Collaborator

nice work just little notes

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.

4 participants