False revoke message in TrustChain

[AlexIvanHoward]

Problem description

The TrustChain viewer displays a "Revoke" message for any VC of which the "comment" field is not empty, even if the VC has in fact been approved.

Steps to reproduce

Steps to reproduce the behavior:

1. Import the testnet 'False revoke in TrustChain' policy via message identifier 1674400089.918995655
2. Open the policy in Dry Run mode.
3. Create a user other than the existing 'Administrator' user.
4. Switch to the user created in step 3 ("Virtual User 1").
5. Select 'PROJECT_DEVELOPER' as the role when prompted.
6. Complete Form A.
7. Switch to the Administrator user.
8. Click on the 'Approve' button for the document which you have just created as Virtual User 1. You should now be prompted for any additional comments to provide to the applicant. Enter "Welcome!" and click 'Ok'.
9. Notice how the status of the document (as shown in column 'Status' of the grid) changed from "NONE" to "APPROVED".
10. Switch back to Virtual User 1.
11. Complete Form B.
12. Switch to the Administrator user.
13. Click on the 'Approve' button for the Form B document which you have just created as Virtual User 1 in step 10 above. You should now be prompted for any additional comments to provide to the applicant. Enter "Great choice!" and click 'Ok'.
14. Notice how the status of the document (as shown in column 'Status' of the grid) changed from "NONE" to "APPROVED".
15. Create a second additional user, Virtual User 2, for the policy in Dry Run and switch to them.
16. Select 'VIEWER' as the role when prompted.
17. You should now be presented with a grid containing one document. Click on the 'View TrustChain' link for the document.
18. Inspect the TrustChain displayed at the bottom of the screen. Notice how the "card" for the Form A VC shows, "Revoked with reason: 'Welcome!'", although the document has in fact been approved. Similarly, notice how the "card" for the Form B VC shows, "Revoked with reason: 'Great choice!'", although the document has in fact been approved. See screenshot below.

Expected behavior

VCs of which the "comment" field is not empty should not be displayed as "Revoked" by default in the TrustChain. Values in the "comment" field of a VC should rather be displayed in an neutral way in the TrustChain, because they can be either acceptance messages or rejection reasons, not just the latter.

Screenshots