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

use freezed package and json_generator to regenerate DTO's #62

Closed
JordyHers opened this issue Jun 2, 2023 · 0 comments
Closed

use freezed package and json_generator to regenerate DTO's #62

JordyHers opened this issue Jun 2, 2023 · 0 comments
Assignees
@i-am-ijaz
Labels
bugfix-branch
Milestone
v.2.0.0

Comments

@JordyHers
Copy link
Contributor

JordyHers commented Jun 2, 2023 •
edited
Loading

Current Behaviour:

The current project class does not have the best approch to generate DTO's.

Expected Behaviour:

implement freezed to have immutable class and generate copy with methods etc...
along with json_generator.

NO generated class should be pushed to github. Make sure you add the extension of the generated files inside .gitignore. and update the readme file to mention to run

flutter pub get && flutter pub run build_runner build --delete-conflicting-outputs

Steps to Reproduce:

NO/

Environment:

Emulator: Pixel 6 Pro
Android version: 12
Android SDK: 33
@JordyHers JordyHers converted this from a draft issue Jun 2, 2023
@JordyHers JordyHers added this to the v.2.0.0 milestone Jun 2, 2023
@i-am-ijaz i-am-ijaz self-assigned this Jun 7, 2023
@i-am-ijaz i-am-ijaz moved this from Backlog to In Progress in Time's up Flutter Jun 7, 2023
i-am-ijaz added a commit that referenced this issue Jun 10, 2023
@i-am-ijaz
fix(#62) : add a guard statement to avoid the empty allMarkers list e…
af3d69c 
…xception
i-am-ijaz added a commit that referenced this issue Jun 10, 2023
@i-am-ijaz
fix(#62) : fixes the models by creating them by using the freezed and…
ad0194f 
… injected
JordyHers added a commit that referenced this issue Jun 14, 2023
@JordyHers
fix(#62) : add parsing extension
b37e97d
JordyHers added a commit that referenced this issue Jun 14, 2023
@JordyHers
fix(#62) : update ci/cd
76a8eba
JordyHers added a commit that referenced this issue Jun 14, 2023
@JordyHers
fix(#62) : update ci/cd to add generate in all jobs steps
e23e7f4
JordyHers added a commit that referenced this issue Jun 14, 2023
@JordyHers
fix(#62) : update ci/cd to add dart fix apply
0b57ba1
JordyHers added a commit that referenced this issue Jun 14, 2023
@JordyHers
fix(#62) : update ci/cd to exclude `run: flutter analyze --exclude=**…
1efa3a8 
…/*.g.dart`
JordyHers added a commit that referenced this issue Jun 14, 2023
@JordyHers
fix(#62) : remove freezed package to the list of linting files
a94d767
JordyHers added a commit that referenced this issue Jun 14, 2023
@JordyHers
fix(#62) : fix issue child is no longer deleted
ad0a92f
@i-am-ijaz i-am-ijaz moved this from In Progress to ready-for-testing in Time's up Flutter Jun 14, 2023
@JordyHers JordyHers moved this from ready-for-testing to Done in Time's up Flutter Jun 17, 2023
@github-project-automation github-project-automation bot moved this from Done to ready-for-testing in Time's up Flutter Jun 17, 2023
@JordyHers JordyHers moved this from ready-for-testing to Done in Time's up Flutter Jun 17, 2023
@JordyHers JordyHers mentioned this issue Jun 17, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@i-am-ijaz i-am-ijaz

Labels
bugfix-branch
Projects
Time's up Flutter
Archived in project
Milestone
v.2.0.0
Development

When branches are created from issues, their pull requests are automatically linked.

bugfix/62-use-freezed-package-and-json_generator-to-regenerate-dtos JordyHers-org/Times-up-flutter
2 participants
@i-am-ijaz @JordyHers