knieper / crltmich

0 stars 0 forks source link

Verifiers are not able to view and set workflow state on the certificate nodes for C1, C2, and D #7

Closed jynxiesue closed 8 years ago

jynxiesue commented 8 years ago

If current logged in user has the umich uniqname corresponding to "field_certificate_mail " (including @umich.ed and/or @med.umich.edu), AND the workflow state is "Needs Verification", that user should be able to change the workflow state to "Needs Review" (seen by user as "I Agree" - this code exists) or "Denied" (seen by user at "I Disagree" - this code exists). Code is at sites/all/modules/custom/crlt/modules/crlt_certificate/crlt_certificate.module

jynxiesue commented 8 years ago

The above functionality used to work but got broken sometime over the summer possibly when grants was deployed.

mbakewel commented 8 years ago

The user can now set the workflow, but the "I agree" and "I disagree" are not showing up. These should be the only two options available - other options like approved, etc. are also available to the user.

anjirlly commented 8 years ago

screen shot 2016-03-22 at 4 03 48 pm

knieper commented 8 years ago

This may be showing if the logged in user has additional permissions (like CRLT staff).

If I am a 'verifer' I see the agree/disagree only (see screenshot) but still have 'needs work' showing. I'll revise to remove 'needs work' screen shot 2016-03-24 at 2 25 22 pm

knieper commented 8 years ago

Now shows just I agree/disagree...example node is 89391 when logged in 'sharonknieper@umich.edu' which is currently set to be the verifier. screen shot 2016-03-24 at 2 34 06 pm

mbakewel commented 8 years ago

tested - all good.