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.

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

Replacing bloc architecture with Atoms #696

Closed
guyluz11 opened this issue Nov 30, 2023 · 1 comment · Fixed by #697
Closed

Replacing bloc architecture with Atoms #696

guyluz11 opened this issue Nov 30, 2023 · 1 comment · Fixed by #697
Assignees
Labels
Improving code organization All the staff that improve code without change to functionality

Comments

@guyluz11
Copy link
Member

Description

Currently we are using bloc which is being implemented in application as well as in presentation.

Our use of bloc is combined with freeze package and all of this logic could be simplified using the architecture of the atom for UI elements and logic to be implemented inside of the pages with stateful widgets.

Let's make the app presentation layer easier to develop and maintain using the Atoms architecture.

Screenshots

No response

@guyluz11 guyluz11 added the Improving code organization All the staff that improve code without change to functionality label Nov 30, 2023
@guyluz11 guyluz11 self-assigned this Nov 30, 2023
@guyluz11 guyluz11 linked a pull request Dec 1, 2023 that will close this issue
@guyluz11
Copy link
Member Author

guyluz11 commented Dec 1, 2023

I have removed bloc package.
Atoms architecture content is almost nonexistent at this point, this process will happen over time.

build runner now runs faster and the app logic is easier to understand, maintenance will join with the architecture of the atom.

Thanks

@guyluz11 guyluz11 closed this as completed Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improving code organization All the staff that improve code without change to functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant