fix: Support for photos in a different language than the default one (green buttons)

[g123k]

Hi everyone,

In the "Fast-track" screen, if the photo is uploaded in a different language than the default one, the button stays gray.
The fix here is to check if at least one photo is pending for the given field.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 8.19%. Comparing base (4d9c7fc) to head (a839715).
Report is 128 commits behind head on develop.

Files	Patch %	Lines
..._app/lib/pages/product/add_new_product_helper.dart	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           develop   #5278      +/-   ##
==========================================
- Coverage     9.54%   8.19%   -1.36%     
==========================================
  Files          325     356      +31     
  Lines        16411   17920    +1509     
==========================================
- Hits          1567    1468      -99     
- Misses       14844   16452    +1608     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[monsieurtanuki]

Hi @g123k!
Your fix is surely an improvement, as it checks not only the current language, but all languages.

I'm afraid it's not 100% good, as the previous version (and yours) only check "transient files" (which are pictures that have not been received by the server yet).
If an image is received by the server, the local product is refreshed (with the new image) and the transient file is removed.
I would suggest using getProductImageLanguages instead.

[g123k]

Hi @g123k! Your fix is surely an improvement, as it checks not only the current language, but all languages.

I'm afraid it's not 100% good, as the previous version (and yours) only check "transient files" (which are pictures that have not been received by the server yet). If an image is received by the server, the local product is refreshed (with the new image) and the transient file is removed. I would suggest using getProductImageLanguages instead.

Thanks for your suggestion, I've checked in the 2 cases and it should be OK now

[g123k]

Hi @g123k! Your fix is surely an improvement, as it checks not only the current language, but all languages.
I'm afraid it's not 100% good, as the previous version (and yours) only check "transient files" (which are pictures that have not been received by the server yet). If an image is received by the server, the local product is refreshed (with the new image) and the transient file is removed. I would suggest using getProductImageLanguages instead.

Thanks for your suggestion, I've checked in the 2 cases and it should be OK now

Oops, there is a merge just after my commit.
Was it the fix you were expecting @monsieurtanuki?