Forum Discussion

Artem_Korneev's avatar
Artem_Korneev
New Contributor
8 years ago

Any comment drops "Approved" state of the code review

In Collaborator 10 any comment drops "Approved" state of the code review back into "Active" state. So, if person A reviewed and signed it off, and then person B leaves some comment, state of the pers...
  • antoinell's avatar
    antoinell
    Visitor
    8 years ago

    We are using Code Collaborator 10.0.10000 and this information does not seem to be correct. We have "Required to Read all Comments" set to No for reviewer in our two active role configurations (used by our templates). There is a third role configuration called "Formal Inspection" but our templates do not use it. "Formal Inspection" has the "Required to Read all Comments" set to Yes for reviewer.

    • AlexeyKryuchkov's avatar
      AlexeyKryuchkov
      SmartBear Alumni (Retired)
      8 years ago

      Hi,

       

      I experimented on the latest version of Collaborator (10.2.10200), so I recommend that you try using it too.

      • samir_patel's avatar
        samir_patel
        New Contributor
        7 years ago

        Hi Alexey,

         

        We are using 11.2.11200 and when I tried to modify this field to be "no" and save, we get an error message "Error: Invalid input: Role Description: You cannot edit role fields because this template is being used in a Review!". So how can we disable and yet keep same Role configuration with exception of "Required to read all comments" to be "No" for reviewer and observer?

         

        Thanks

        Samir