Skip to content

Set default destinyInformation to empty list in AtsLoadingParamsFormInput#194

Merged
ManuelRomeroA merged 1 commit into
mainfrom
development
May 14, 2026
Merged

Set default destinyInformation to empty list in AtsLoadingParamsFormInput#194
ManuelRomeroA merged 1 commit into
mainfrom
development

Conversation

@ManuelRomeroA
Copy link
Copy Markdown
Contributor

This pull request updates the AtsLoadingParamsFormInput model to make the destinyInformation field non-nullable and ensures it always defaults to an empty list. This change affects the model definition, code generation, and JSON serialization/deserialization to improve type safety and avoid potential null-related bugs.

Model and API changes:

  • Changed the destinyInformation field in AtsLoadingParamsFormInput from a nullable List<AtsDestinyInformationInput>? to a non-nullable List<AtsDestinyInformationInput> with a default value of an empty list using @Default([]). This ensures the field is always present and never null. [1] [2]

Code generation and copyWith methods:

  • Updated the generated copyWith methods and constructors in lib/src/ats/ats.freezed.dart to reflect the new non-nullable type and default value for destinyInformation. This includes changing method signatures and default parameter values. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

Serialization/Deserialization:

  • Modified the generated JSON serialization and deserialization logic to ensure destinyInformation is always deserialized as a non-null list (defaulting to [] if missing or null in the JSON). [1] [2]

Documentation and versioning:

  • Updated the CHANGELOG.md to document this change and bumped the package version to 3.8.13 in pubspec.yaml. [1] [2]

@github-actions
Copy link
Copy Markdown

📋 Changelog Summary

This PR includes the following changes:

✨ Features


🤖 Auto-generated by changelog workflow

@github-actions
Copy link
Copy Markdown

📊 Test Coverage Report

🎯 Dart/Flutter

Metric Value
Coverage 11.4%
Lines covered 14936 / 450
Lines missing -14486

🤖 Generated by coverage workflow

@ManuelRomeroA ManuelRomeroA merged commit de32615 into main May 14, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant