feat: 4947 - added local "last access" timestamp for products

[monsieurtanuki]

What

• As @LuanRoger suggested, I've added a local "last access" timestamp for products.
• With this timestamp, we'll be able to refresh the most interesting products first, for instance when we switch app languages.

Part of

• Add metadata about local Product, for better refresh flexibility #4947

Files

New file:

• dao_product_last_access.dart: Table that stores the local last access timestamp for a product.

Impacted files:

• local_database.dart: new version of the database, because of the new "product last access" table
• new_product_page.dart: added a call to refresh the product "last access" timestamp
• offline_data_page.dart: added a call to delete the product "last access" timestamps
• product_model.dart: added a call to refresh the product "last access" timestamp
• pubspec.lock: wtf

[codecov-commenter]

Codecov Report

Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (85784fa) 9.62% compared to head (5a45acd) 9.61%.

Files	Patch %	Lines
...ooth_app/lib/database/dao_product_last_access.dart	0.00%	9 Missing ⚠️
...smooth_app/lib/pages/product/new_product_page.dart	0.00%	3 Missing ⚠️
...ckages/smooth_app/lib/pages/offline_data_page.dart	0.00%	2 Missing ⚠️
...ckages/smooth_app/lib/database/local_database.dart	0.00%	1 Missing ⚠️
...th_app/lib/pages/product/common/product_model.dart	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           develop   #4969      +/-   ##
==========================================
- Coverage     9.62%   9.61%   -0.01%     
==========================================
  Files          323     324       +1     
  Lines        16267   16282      +15     
==========================================
  Hits          1565    1565              
- Misses       14702   14717      +15     

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

[teolemon]

@M123-dev @g123k @AshAman999 could you have a look ?

[M123-dev]

A small naming thing, besides that good idea 👍🏼

[M123-dev]

The name "LastAccess" could be confusing, we should clarify somewhere that its the last server access, not last local access to the product

[monsieurtanuki]

Thank you @M123-dev for your review!

The name "LastAccess" could be confusing, we should clarify somewhere that its the last server access, not last local access to the product

/// Table that stores the local last access timestamp for a product.
class DaoProductLastAccess extends AbstractSqlDao {
// ...
  static const String COLUMN_LAST_ACCESS = 'last_access';

I did specify it's the "local last access timestamp for a product".

Now that the table is created, if someone finds a better naming for the class (DaoProductLatestLocalAccess instead?) or the column (COLUMN_LATEST_LOCAL_ACCESS instead ?), why not, it will have no major impact. As long as we don't change the SQL table and columns names.