Feature: Tags history

[pylipp]

• Add temporal columns to TagsRelation
• Update TagsRelation business logic to set temporal fields when assigning or removing tags
• Update DB seed

[pylipp]

2.0/dropapp: implement temporal awareness about adding/removing tags

[HaGuesto]

code reviewed passed

feature review

We move the feature review to when INserts and Deletions are also covered, because we need an updated init.sql here first.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.59%. Comparing base (0317b05) to head (14f14ab).
Report is 73 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1472   +/-   ##
=======================================
  Coverage   85.59%   85.59%           
=======================================
  Files         232      232           
  Lines       21710    21727   +17     
  Branches     1916     1920    +4     
=======================================
+ Hits        18582    18597   +15     
- Misses       3088     3090    +2     
  Partials       40       40           

Flag	Coverage Δ
backend	99.02% <ø> (-0.05%)	⬇️
frontend	82.35% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[HaGuesto]

functional review

@pylipp
There is only one case where I'd probably change something. I would not show Box history items about assigning tags on the timestamp when the box is created. We also do not show the selected product at the moment of creation in the box history and this would end up confusing when tags are assigned before the box is created.

Therefore, I'd remove the assign tags entries from the box history when the box is created.

[pylipp]

I agree that it might be confusing to see the created-record entries as more recent in the history. I thought this might be solved by grouping history entries with identical timestamp (tagged you in trello).

If you think this is not sufficient, I'll try to make the assigned-tag entries that belong to creation disappear. I think the easiest way is to not set the created-fields of TagsRelations when creating a box. Then these fields would only be set if an existing box is updated with tags.