Skip to content

Commit

Permalink
Merge pull request #3368 from project-koku/release/prod-stable.2524
Browse files Browse the repository at this point in the history
Deployment commit for prod-stable
  • Loading branch information
dlabrecq authored Aug 28, 2023
2 parents 13ffbbd + cf8e1d5 commit 91140d7
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
3 changes: 1 addition & 2 deletions scripts/release-branch.sh
Original file line number Diff line number Diff line change
Expand Up @@ -74,9 +74,8 @@ pullRequest()
echo "\n*** Pushing $NEW_BRANCH..."
git push -u origin HEAD

COMMIT=`git rev-parse HEAD`
TITLE="Deployment commit for $BRANCH"
BODY="Merged $REMOTE_BRANCH branch to $BRANCH. Use commit $COMMIT to update namespace \`ref\` in app-interface repo"
BODY="Merged $REMOTE_BRANCH branch to $BRANCH. Use latest commit to update namespace \`ref\` in app-interface repo. Don't use merge commit, SHAs must be unique when images are created for each branch."

gh pr create -t "$TITLE" -b "$BODY" -B $BRANCH
}
Expand Down
4 changes: 2 additions & 2 deletions src/components/permissions/permissions.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ const PermissionsBase: React.FC<PermissionsProps> = ({
const ocp = hasOcpAccess(userAccess);
const rhel = isFinsightsFeatureEnabled && hasRhelAccess(userAccess);
const ros = isRosFeatureEnabled && hasRosAccess(userAccess);
const settings = isSettingsFeatureEnabled && hasSettingsAccess(userAccess);
const settings = (costModel || hasSettingsAccess(userAccess)) && isSettingsFeatureEnabled;

switch (pathname) {
case formatPath(routes.explorer.path):
Expand Down Expand Up @@ -102,7 +102,7 @@ const PermissionsBase: React.FC<PermissionsProps> = ({
case formatPath(routes.rhelDetailsBreakdown.path):
return rhel;
case formatPath(routes.settings.path):
return costModel || settings;
return settings;
default:
return false;
}
Expand Down
2 changes: 1 addition & 1 deletion src/store/costModels/actions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ export const redirectToCostModelFromSourceUuid = (source_uuid: string, router: R
const uuid = res.data.data[0].uuid;
router.navigate(
`${formatPath(
selectIsSettingsFeatureEnabled(getState()) ? routes.settings.path : routes.costModelsDetails.path
selectIsSettingsFeatureEnabled(getState()) ? routes.costModel.basePath : routes.costModelsDetails.path
)}/${uuid}`
);
})
Expand Down

0 comments on commit 91140d7

Please sign in to comment.