-
Notifications
You must be signed in to change notification settings - Fork 17
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
Fix add to report #2295
Fix add to report #2295
Conversation
@@ -55,13 +55,14 @@ export const AddToReportMenu: React.FC<IAddToReportMenuProps> = ({ content, onCl | |||
|
|||
if (!report.instances.length) { | |||
// Fetch the report because the one in memory has no instances. | |||
currentReport = await getReport(report.id); | |||
currentReport = await getReport(report.id, true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was causing issues on the first time adding content to a report.
@@ -72,7 +72,7 @@ spec: | |||
buildVar="${1/\-/\_}" | |||
# convert to uppercase | |||
buildVar="${buildVar^^}" | |||
BUILD_${buildVar}=true | |||
BUILD=BUILD_${buildVar}=true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed bug I introduced. However, the test pipeline still doesn't work for some reason.
@@ -410,6 +410,7 @@ Auth__Keycloak__Secret={YOU WILL NEED TO GET THIS FROM KEYCLOAK} | |||
Auth__OIDC__Token=/realms/mmi/protocol/openid-connect/token | |||
|
|||
Service__ApiUrl=http://host.docker.internal:$portApi/api | |||
Service__DataLocation=Openshift |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes it easier for new resources to get the syndication service running
An initial fix for add content to report. A more complex fix needs to be implemented to sync multiple tabs, but this should resolve a few errors.