You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Whitehall stores link_check_reports for each Edition, as a has_many relationship. This was added in 2014 with little explanation.
In the UI, we only ever expose the most recent link checker report, so these historical reports are redundant. A has_one relationship, storing only the most recent report, would be more appropriate.
NB: there is a spike capturing some of the work involved in paying down this technical debt.
Consequences
Makes Whitehall unnecessarily more complex to write code in.
Cause
Whitehall stores
link_check_reports
for each Edition, as ahas_many
relationship. This was added in 2014 with little explanation.In the UI, we only ever expose the most recent link checker report, so these historical reports are redundant. A
has_one
relationship, storing only the most recent report, would be more appropriate.NB: there is a spike capturing some of the work involved in paying down this technical debt.
Consequences
Edition.only_broken_links
method.The text was updated successfully, but these errors were encountered: