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

feat: #2501 - added "origins" in edit product page #2571

Merged
merged 3 commits into from
Jul 11, 2022

Conversation

monsieurtanuki
Copy link
Contributor

Impacted files:

  • app_en.arb: added 2 labels for "origins"
  • app_fr.arb: added 2 labels for "origins"
  • edit_product_page.dart: added an item for "origins"
  • product_query.dart: added field "origins" to the list of the product fields we retrieve
  • pubspec.lock: wtf
  • pubspec.yaml: upgraded to openfoodfacts: ^1.22.0, which has new product field "origins"
  • simple_input_page_helpers.dart: implementation for "origins" of a simple input page item

What

  • Added "origins" in edit product page.
  • Little problem: as "origins" was not part of the product fields we retrieved, the products currently in the local database don't have this field. Refresh is needed.

Screenshot

edit product page origins
Capture d’écran 2022-07-11 à 08 13 47 Capture d’écran 2022-07-11 à 08 14 25

Fixes bug(s)

Impacted files:
* `app_en.arb`: added 2 labels for "origins"
* `app_fr.arb`: added 2 labels for "origins"
* `edit_product_page.dart`: added an item for "origins"
* `product_query.dart`: added field "origins" to the list of the product fields we retrieve
* `pubspec.lock`: wtf
* `pubspec.yaml`: upgraded to `openfoodfacts: ^1.22.0`, which has new product field "origins"
* `simple_input_page_helpers.dart`: implementation for "origins" of a simple input page item
@monsieurtanuki monsieurtanuki requested a review from a team as a code owner July 11, 2022 06:16
@monsieurtanuki monsieurtanuki requested a review from teolemon July 11, 2022 06:17
@codecov-commenter
Copy link

Codecov Report

Merging #2571 (8e9881b) into develop (2ea0da3) will decrease coverage by 1.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #2571      +/-   ##
==========================================
- Coverage     8.86%   7.55%   -1.32%     
==========================================
  Files          161     210      +49     
  Lines         6623   10103    +3480     
==========================================
+ Hits           587     763     +176     
- Misses        6036    9340    +3304     
Impacted Files Coverage Δ
...kages/smooth_app/lib/widgets/attribute_button.dart 0.00% <0.00%> (-92.00%) ⬇️
...s/smooth_app/lib/data_models/user_preferences.dart 8.73% <0.00%> (-23.57%) ⬇️
packages/smooth_app/lib/themes/smooth_theme.dart 60.00% <0.00%> (-22.98%) ⬇️
...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart 15.29% <0.00%> (-18.92%) ⬇️
...mooth_app/lib/data_models/product_preferences.dart 24.65% <0.00%> (-6.78%) ⬇️
packages/smooth_app/lib/main.dart 14.16% <0.00%> (-3.73%) ⬇️
.../smooth_app/lib/pages/onboarding/welcome_page.dart 0.00% <0.00%> (-3.13%) ⬇️
.../smooth_app/lib/pages/onboarding/scan_example.dart 0.00% <0.00%> (-2.28%) ⬇️
...p/lib/pages/onboarding/consent_analytics_page.dart 0.00% <0.00%> (-1.57%) ⬇️
...smooth_app/lib/pages/product/new_product_page.dart 0.00% <0.00%> (-0.88%) ⬇️
... and 219 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b16d6d6...8e9881b. Read the comment docs.

@teolemon
Copy link
Member

teolemon commented Jul 11, 2022

The last of Eco-Score components 👌
Can we add travel_explore as an icon?

Impacted files:
* `simple_input_page_helpers.dart`
@monsieurtanuki
Copy link
Contributor Author

Done:
Capture d’écran 2022-07-11 à 09 17 31

Co-authored-by: Pierre Slamich <pierre.slamich@gmail.com>
@teolemon
Copy link
Member

Thank you very much @monsieurtanuki

@teolemon teolemon merged commit 2b60cb5 into openfoodfacts:develop Jul 11, 2022
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.

Implement simplified origin input
4 participants