Skip to content

Make the type detector even smarter #124

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

Closed
vanlooverenkoen opened this issue Dec 19, 2022 · 0 comments · Fixed by #138
Closed

Make the type detector even smarter #124

vanlooverenkoen opened this issue Dec 19, 2022 · 0 comments · Fixed by #138
Assignees
Labels
P0 Highest prio P2 Medium prio

Comments

@vanlooverenkoen
Copy link
Contributor

Project:
  path: webservice/projects
  equals_and_hash_code: false
  properties:
    projectId: int
    name: String
    apiKey: String?
    activated: bool
    locales:
      type: array
      items:
        type: WritorLocale
      required: false
      include_if_null: false

could be this:

Project:
  path: webservice/projects
  equals_and_hash_code: false
  properties:
    projectId: int
    name: String
    apiKey: String?
    activated: bool
    locales:
      type: List<WritorLocale>?
      include_if_null: false

because include_if_null:false was added it is not possible to use List<WritorLocale>? anymore

@vanlooverenkoen vanlooverenkoen added the P2 Medium prio label Dec 19, 2022
@vanlooverenkoen vanlooverenkoen added the P0 Highest prio label Aug 10, 2023
jorre127 added a commit that referenced this issue Aug 14, 2023
jorre127 added a commit that referenced this issue Aug 14, 2023
jorre127 added a commit that referenced this issue Aug 14, 2023
jorre127 added a commit that referenced this issue Aug 14, 2023
@jorre127 jorre127 moved this from Todo to In Progress in icapps Flutter OSS Aug 14, 2023
@jorre127 jorre127 self-assigned this Aug 14, 2023
@jorre127 jorre127 moved this from In Progress to In Review in icapps Flutter OSS Aug 14, 2023
jorre127 added a commit that referenced this issue Aug 21, 2023
jorre127 added a commit that referenced this issue Aug 21, 2023
jorre127 added a commit that referenced this issue Aug 21, 2023
jorre127 added a commit that referenced this issue Aug 22, 2023
jorre127 added a commit that referenced this issue Aug 22, 2023
jorre127 added a commit that referenced this issue Aug 29, 2023
jorre127 added a commit that referenced this issue Aug 29, 2023
jorre127 added a commit that referenced this issue Aug 29, 2023
jorre127 added a commit that referenced this issue Aug 30, 2023
jorre127 added a commit that referenced this issue Aug 30, 2023
vanlooverenkoen added a commit that referenced this issue Aug 30, 2023
@github-project-automation github-project-automation bot moved this from In Review to Done in icapps Flutter OSS Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 Highest prio P2 Medium prio
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants