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

feature/#210 - add nutrient order and list #271

Merged
merged 2 commits into from
Oct 19, 2021

Conversation

monsieurtanuki
Copy link
Contributor

New files:

  • ordered_nutrient_test.dart: Tests related to OrderedNutrient and OrderedNutrients
  • OrderedNutrient.dart: Nutrient, as a hierarchically ordered and localized entity.
  • OrderedNutrients.dart: Nutrients, as hierarchically ordered and localized entities.

Generated files (from JsonSerializable):

  • OrderedNutrient.g.dart
  • OrderedNutrients.g.dart

New files:
* `ordered_nutrient_test.dart`: Tests related to `OrderedNutrient` and `OrderedNutrients`
* `OrderedNutrient.dart`: Nutrient, as a hierarchically ordered and localized entity.
* `OrderedNutrients.dart`: Nutrients, as hierarchically ordered and localized entities.

Generated files (from JsonSerializable):
* `OrderedNutrient.g.dart`
* `OrderedNutrients.g.dart`
…oring

Thanks to @peterwvj and @jasmeet0817!

Impacted files:
* `ordered_nutrient_test.dart`: added a comment, removed commented code
* `OrderedNutrient.dart`: simplified around JsonSerializable
* `OrderedNutrient.g.dart` (generated from JsonSerializable)
* `OrderedNutrients.dart`: simplified around JsonSerializable
* `OrderedNutrients.g.dart` (generated from JsonSerializable)
Copy link
Collaborator

@peterwvj peterwvj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@monsieurtanuki monsieurtanuki merged commit 787bdd4 into openfoodfacts:master Oct 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants