Cécile Madjar

@nicolasbrossard I think this is by design that you can see all DICOM archives. The reason for it is in case an archive is not associated to a site (aka...

@nicolasbrossard smart :). Is that something you can work on eventually? Otherwise we'll find someone else to do it. Just let us know.

@AlexandraLivadas could you please rebase so that Travis can run on it? Thank you!

@ridz1208 to re-review and reply @AlexandraLivadas

@christinerogers I think this is more a question for @driusan or @ridz1208. I am mainly the note taker when I assign on GitHub, lol

@driusan eventually, we will need to modify the mri_violations module to display those new fields. I see that image_type is not in the mri_violations either so I was thinking of...

@driusan @nicolasbrossard before I move forward with this work, could you take a look at the new SQL structure and let me know if the changes make sense? I'd like...

@driusan thank you for the review!! See my answers to your comments. Let me know your thoughts. We can also discuss during the imaging call. Thanks!

@driusan ready for another round of review. All your comments have been replied to or taken care of in the last few commits. I also fixed the dashboard automated tests...

@driusan @kongtiaowang I don't understand where the error comes from in the tests. Would you mind taking a look?