Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature -> Develop | Added BLoc State Management #48

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Feature -> Develop | Added BLoc State Management #48

wants to merge 1 commit into from

Conversation

we-lokesh-m001
Copy link
Collaborator

@we-lokesh-m001 we-lokesh-m001 commented Mar 3, 2025 •
edited
Loading

[Task #58 - Flutter > VaahFlutter > Core > State Management Migration To BLoc]
(https://team.webreinvent.com/backend/collab/project/vaahflutter-service-bell/tasks/58)

Time invested: hh:mm format | Billable: 15:10 | Non-Billable: 00:00

Problem Statement

  1. To migrate GetX state-management to BLoc.

Describe The Fix/ Solution You Implemented

Removed the GetX logic throughout the app and implemented the flutter_bloc & bloc state-management along with managing the UI & alert dialog box contexts.

Proof of your testing (Demo link or video links or image links)

Dependencies

  • Updated: None
  • Added:
    • bloc: ^9.0.0
    • flutter_bloc: ^9.0.0
    • equatable: ^2.0.7
  • Removed:
    • get: ^4.6.6

Merge Request Checklist

  • I have performed a self-review of my code
  • My code does follow industry standards
  • My code doesn't produce warnings/ errors for dart analyzer
  • New and existing tests pass locally with my changes (No tests are there as of now)
  • The code modified as part of this PR has been covered with tests
  • My code is properly formatted
  • I've followed the proper naming conventions
  • I have rebased the feature on the latest develop
  • I have run flutter run after the rebase
  • I have added new dependencies/ updated old dependencies
  • I have read all the comments & notes in wireframe and verified that I did take care of that
  • verify that the UI must match with design & wireframe if available
  • I have updated the version/ build (x.x.x+xxx)
  • I've verified that the latest commit of develop exists in your feature branch after rebase.
  • My fix achieves 100% of what is required
  • Can enhance the solution in the future because better implementation could be made

@we-lokesh-m001
added: migration to BLoc from GetX with following singleton approach
6d5089d 
refactored: environment_configs access & bloc logic separated


fix: all imports


removed: unnecessary comments & EnvBloc updation usecase


added: navigation with navigator


fix: imports
@we-lokesh-m001 we-lokesh-m001 self-assigned this Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@we-prajapati-c001 we-prajapati-c001

Assignees

@we-lokesh-m001 we-lokesh-m001

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@we-lokesh-m001