faragom
commented
3 years ago Well, it is always better if a continuity solution is implemented. For example:
- The button is disabled and cannot be pushed (with a side note explaining why and what should be done to have it activated)
- The button triggers the process to reach the conditions (the DID-auth is done and the session DataStore is cleared to avoid the security issue)
But given the situation, it will be the easiest solution for now. The client can handle it by prompting a message explaining that for it to be available, the user must close the session and do did-auth at the beginning.
El mar, 1 dic 2020 a las 16:26, mvj66 (notifications@github.com) escribió:
Hello @faragom https://github.com/faragom ,
In order to fully implement the Issue VC from Dashboard https://github.com/EC-SEAL/interface-specs/blob/master/images/UC5_01_Issue_VC_v4.png, the option for "Issue Verifiable Credentials" in the Dashboard would work only if the user accesses by SSI and not by PDS.
Do you think it would be OK that option was available always, and the APIGW returned an error if the access was by PDS? The ssiWallet https://github.com/EC-SEAL/interface-specs/wiki/session-variables variable would be read for checking it.
Thanks in advance.
FYI: @Raul14 https://github.com/Raul14 , @albbasdur https://github.com/albbasdur , @ross-little https://github.com/ross-little
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/EC-SEAL/interface-specs/issues/5, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG4HJ64QJ26WUKJUWSVC6RTSSUDMBANCNFSM4UJFVIZQ .
-- Francisco José Aragó Monzonís mitsurugisan@gmail.com
Hello @faragom ,
In order to fully implement the Issue VC from Dashboard, the option for "Issue Verifiable Credentials" in the Dashboard would work only if the user accesses by SSI and not by PDS.
Do you think it would be OK that option was available always, and the APIGW returned an error if the access was by PDS? The ssiWallet variable would be read for checking it.
Thanks in advance.
FYI: @Raul14 , @albbasdur , @ross-little