Skip to content

Conversation

@pabel-rh
Copy link
Member

@pabel-rh pabel-rh commented Jan 19, 2026

@rhdh-bot
Copy link
Collaborator

rhdh-bot commented Jan 19, 2026

@pabel-rh pabel-rh requested a review from imykhno January 19, 2026 06:48

To enable aggregation, you must perform the following steps:

. Define relationships: Ensure sub-entities include the partOf relationship in their definition.
Copy link

Choose a reason for hiding this comment

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

I think that Define relationships is right step. However I would like to propose to change the description to: Ensure user owns the entities themselves or when the user is a member of a group which owns entities

Comment on lines 14 to 17

. Apply metadata labels: Add specific labels to the entity metadata to help the backend identify sub-entities.

. Enable aggregation: Toggle the aggregation setting in the parent entity view to display the consolidated score.
Copy link

Choose a reason for hiding this comment

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

I am not sure that I understand correctly the Apply metadata labels and Enable aggregation points. Could you explain why them important to add?

Copy link

Choose a reason for hiding this comment

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

I would recommend to add: Define permissions: Ensure user have scorecard permissions

Copy link
Member Author

Choose a reason for hiding this comment

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

I was thinking of removing this "Procedure" section in the assembly as the next procedural module talks about it.

Copy link

Choose a reason for hiding this comment

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

I would like to ask where this information was obtained?

Copy link

Choose a reason for hiding this comment

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

I would like to ask where this information was obtained?

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants