Wyclif Luyima

icon company OpenMRS icon location Austin Texas

Results 13 comments of Wyclif Luyima

TRUNK-6081: Add not null constraint to encounter_diagnosis.dx_rank

Looks good

[14.0][IMP] auth_oidc: Logout the user from the auth provider

@ap-wtioit when do you plan to have this merged?

[14.0][IMP] auth_oidc: Logout the user from the auth provider

@sbidoul do you mind becoming the second reviewer/approver on this PR?

[14.0][IMP] auth_oidc: Logout the user from the auth provider

I see that this was added to the 14.0 milestone, it would be great to forward port the changes to later milestones too i.e. 15, 16 and 17

[14.0][IMP] auth_oidc: Logout the user from the auth provider

@sbidoul tests have been added

[14.0][IMP] auth_oidc: Logout the user from the auth provider

@sbidoul @ap-wtioit tests were added, did you get a chance to take a look?

[14.0][IMP] auth_oidc: Logout the user from the auth provider

Done and squashed commits

[14.0][IMP] auth_oidc: Logout the user from the auth provider

Hi @ap-wtioit, is this waiting on anything to be merged?

Add support to skip the logout confirmation in the auth provider

@ap-wtioit, @sbidoul our team wishes to work on this and issue a PR.

[17.0][IMP] Move all auto login logic to the backend

We have been using this feature for a while and we have not experienced that issue but I will test and verify.