Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add assigment of access items via permissions #2603

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

ryzheboka
Copy link
Contributor


Release Notes:


For the submitter:

Verified by the reviewer:

  • Commit message format → (Closes) #<Issue Number>: Your commit message.
  • Submitter's update to documentation is sufficient
  • SonarCloud analysis meets our standards
  • Update of the current release notes reflects changes
  • PR fulfills the ticket
  • Edge cases and unwanted side effects are tested
  • Readability

Copy link
Contributor

@holgerhagen holgerhagen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is still a major problem, when applying a permission as access item to a workbasket. The URL "http://localhost:8080/taskana/api/v1/access-ids?search-for=sach" returns the full DN instead of the permission name only. I haven't found out yet

UPDATE: I believe the reason is that for those 'permissions' the permission attribute is missing. Since I've left another comment there the problem should disappear once the ldif file is correct.

@ryzheboka ryzheboka force-pushed the TSK-2563 branch 20 times, most recently from 86845f8 to 2e5c37a Compare July 24, 2024 09:48
@ryzheboka ryzheboka force-pushed the TSK-2563 branch 4 times, most recently from ded37a1 to a963e73 Compare July 26, 2024 07:59
Co-authored-by: SebastianRoseneck <[email protected]>
Co-authored-by: ryzheboka <[email protected]>
@ryzheboka ryzheboka merged commit 31100a5 into Taskana:master Jul 26, 2024
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants