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

Sprint13 staging #124

Merged
merged 5 commits into from
Dec 20, 2023
Merged

Sprint13 staging #124

merged 5 commits into from
Dec 20, 2023

Conversation

kaladay
Copy link

@kaladay kaladay commented Dec 18, 2023

No description provided.

kaladay and others added 5 commits December 5, 2023 12:51
This gets the permissions structure to be compliant by having the module name as part of the permission name.
Without this, potential cross-module conflicts exist.

This has a side-effect of `workflow.workflow.` as a permission.
The additional workflow represents the `workflow` path.
This is not removed to avoid potential naming conflicts as well.
…to one mapping of the id to permissions.

It simply makes more sense to use one to one mapping of the id to the permission.
If there is an 's' in the id, then use an 's' in the permission.
This is preventing proper deployment in OKAPI.
The `mod-camunda` path should probably also be removed from the OKAPI settings.
Issue 113: Make changes needed to successfully register deploy module.
@kaladay kaladay merged commit 9d29ef7 into main Dec 20, 2023
@kaladay kaladay deleted the sprint13-staging branch December 20, 2023 20:42
rmathew1011 pushed a commit that referenced this pull request Nov 12, 2024
Add support for Jacoco to allow for local coverage reports.
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.

2 participants