ILIAS icon indicating copy to clipboard operation
ILIAS copied to clipboard

(bugfix#30128): fixed container name, added rol_id to distinguish roles with same names

Open utesche opened this issue 4 years ago • 9 comments

The associated rol_id is enclosed in colons and prefixed to lookup list entries. If the choosen (or typed) selection begins with such a word (digits enclosed in colons), it is used directly as the rol_id for the assignment.

Perhaps the list of current assignments should be tempered accordingly... not included in this PR.

utesche avatar Feb 22 '21 11:02 utesche

Hi @smeyer-ilias

we stumbled upon this PR in our TB meeting. If you don't have any objections, please merge this PR or provide feedback otherwise.

Best regards!

kergomard avatar Sep 21 '21 14:09 kergomard

Hi @smeyer-ilias As Technical Board, we regularly check for pull requests that have been open for a long time. As the assignee, if you don't have any objections, please merge this PR or provide feedback otherwise.

Best regards!

chfsx avatar Nov 30 '21 10:11 chfsx

Hi @smeyer-ilias

we stumbled upon this PR in our TB meeting. Any updates on this?

Best regards!

mjansenDatabay avatar Feb 08 '22 12:02 mjansenDatabay

Hi @smeyer-ilias

As Technical Board, we regularly check for pull requests that have been open for a long time. As the assignee, if you don't have any objections, please merge this PR or provide feedback otherwise.

Best regards!

Amstutz avatar Apr 05 '22 07:04 Amstutz

Hi @smeyer-ilias As Technical Board, we regularly check for pull requests that have been open for a long time. Any Updates on this?

Best regards!

Amstutz avatar May 03 '22 11:05 Amstutz

Hi @smeyer-ilias As Technical Board, we regularly check for pull requests that have been open for a long time. Any Updates on this?

Best regards!

Amstutz avatar May 17 '22 09:05 Amstutz

Hi @smeyer-ilias As Technical Board, we regularly check for pull requests that have been open for a long time. Any Updates on this?

Best regards!

Amstutz avatar Jun 14 '22 08:06 Amstutz

Hi @smeyer-ilias As Technical Board, we regularly check for pull requests that have been open for a long time. Any Updates on this?

Best regards!

Amstutz avatar Aug 09 '22 10:08 Amstutz

Sorry, for the delay. I will check and test this PR after my vacation.

smeyer-ilias avatar Aug 12 '22 12:08 smeyer-ilias

Dear PR, please accept my sincere apologies for not congratulating you on your 2nd birthday!

All joking aside... @pascalseeland can you give an assessment of whether you can look at this PR, whether it needs adjustments or how to proceed with it? thank you very much!

chfsx avatar Jan 24 '23 12:01 chfsx

Hi @pascalseeland

As Technical Board, we regularly check for pull requests that have been open for a long time. Any Updates on this? Note, that you can also close this, if you are not able or if you not have the ressources to look into it in detail.

Best regards!

klees avatar Jul 11 '23 11:07 klees

Adding reverse pointer: see comment dated 07 AUG 2023 by @utesche on a related matter, the Role assignment of local roles for OpenID Connect (OIDC) Authentication.

nicoroeser avatar Aug 07 '23 12:08 nicoroeser

As mentioned in the bug report, I proposed a different approach to this issue.

pascalseeland avatar Sep 18 '23 14:09 pascalseeland