refactor(app): refactor ODD protocol card copy when run data is not "ok"

[mjhuff]

Closes RQA-3851

Overview

When protocol cards have not ok run data, we can't retrieve the completedAt timestamp, and the fallback has been the current date, which is confusing copy. After speaking with Design, the plan is to remove copy if we can't retrieve the completedAt timestamp.

As an aside, I did investigate the suffix comment left in the code. After playing around with it a bit and reading the docs, turns out we do need to keep suffix: true.

Test Plan and Hands on Testing

• See pic.

Changelog

• ODD recently run protocol cards no longer show the current time when the last run completed time is unknown.

Risk assessment

very low