Skip to content

Fix a value in license compliance MR widget endpoing

Tetiana Chupryna requested to merge 366930-fix-regresssion-in-be into master

What does this MR do and why?

This is a fix for the issue found here !91806 (comment 1019281370)

In !87150 (merged) approval_required data wasn't propagated to the service. That happened because our caching mechanism doesn't allow us to pass additional arguments. But while working on the fix I discovered that we still pass merge request id and can use this value to get the information needed into the service.

Note: feature is behind feature flag so we don't need changelog.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #366930 (closed)

Merge request reports