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

Change App folder structuring #134

Open
Mohitmadhav opened this issue Oct 8, 2021 · 2 comments
Open

Change App folder structuring #134

Mohitmadhav opened this issue Oct 8, 2021 · 2 comments
Labels
easy Easy level issue good first issue good issue for first-time contributors normal-priority Something that has a normal priority refactor Improve code quality

Comments

@Mohitmadhav
Copy link
Member

Description

  • Change the app root structuring by distributing common files separately.
  • For instance, look into books and base folder in lib/services. We need to use and remodify them.
  • Separate models and providers. Like currently there are 2 files - book.dart, and books.dart...need to structure them according to separate folders.
  • Similarly, we need to check for all the dart files to maintain consistency.
  • You can rename the files if they don't have suitable names.

Steps to Reproduce

Please provide detailed steps for reproducing the issue.

  1. Use snake_camel case for file naming and lower case for folder naming
  2. No 2 files or folders should have the same name (Modify if any)
  3. No dependencies should be changed. We just need to refactor the code and folders

Additional data

Make modifications according to standard practices, and we can have further discussions here or in #91

@Mohitmadhav Mohitmadhav added easy Easy level issue good first issue good issue for first-time contributors Hacktoberfest For Hacktoberfest 2021 normal-priority Something that has a normal priority refactor Improve code quality labels Oct 8, 2021
@lalit-commits
Copy link
Contributor

Hey @Mohitmadhav ,I would like to work on this issue ,please assign it to me.

@Mohitmadhav Mohitmadhav removed the Hacktoberfest For Hacktoberfest 2021 label May 31, 2022
@Nishanth-123
Copy link

Nishanth-123 commented Dec 31, 2022

Hey @Mohitmadhav and @ankitoscar, I would like to work on this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
easy Easy level issue good first issue good issue for first-time contributors normal-priority Something that has a normal priority refactor Improve code quality
Projects
None yet
Development

No branches or pull requests

3 participants