Change label "Virus N status" to "RT-PCR"

added Doing label

added Covid-19 label

mentioned in merge request !250 (merged)

the change done seems to be from Virus N status to Virus N RT-PCR. not in line with the title of the ticket. Also, shouldn't it also be done on the subject details page?

I'm spending a lot of time trying to solve one issue. Maybe you can help me with this one @valentin.groues

I solved the issue you mention, changing the verbose name. See commit 3ea0403f

But I still have a problem with the value. In visits/details/<id> and appointments/edit/<id> it should put inconclusive instead of N/A for those virus_visit_X that are None but their corresponding date virus_visit_X_updated has a date. That's how we are encoding the 4 different values with the boolean field.

N/A means we still don't have any result.
Inconclusive is that the result we have is inconclusive (boolean field is null, but corresponding date is populated)
Yes is that result was positive
No is that result was positive

As you can see, this should read Inconclusive for the first virus visit because we have the date. Can you take a look on this?. These fields should be read-only, so, I understand we could have a normal label instead of a select. But it seems to be automatic somehow. I checked StudySubjectEditForm and StudySubjectForm but I don't find the right way to change this. Ideally, it should be done in a way that if in the future is editable, the field will be encoded and somehow warn that the date will be lost too.

If we could also improve the presentation (html template) so that each virus visit and date go together in pairs, that would be perfect.