Skip to content

Commit

Permalink
UI – 2 small unreleased bug fixes for software install policy automat…
Browse files Browse the repository at this point in the history
…ions UI (#22181)

## 2 small fixes for issues found in QA

- Capitalize word in tooltip
- Correctly render error message, with correctly colored icon

- [x] Manual QA for all new/changed functionality

---------

Co-authored-by: Jacob Shandling <[email protected]>
  • Loading branch information
jacobshandling and Jacob Shandling committed Sep 17, 2024
1 parent 518a4a1 commit f135d5b
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { InjectedRouter } from "react-router";
import PATHS from "router/paths";
import { AppContext } from "context/app";
import { NotificationContext } from "context/notification";
import { SoftwareInstallStatus, ISoftwarePackage } from "interfaces/software";
import { ISoftwarePackage } from "interfaces/software";
import softwareAPI from "services/entities/software";

import { buildQueryStringFromParams } from "utilities/url";
Expand Down Expand Up @@ -104,7 +104,7 @@ const STATUS_DISPLAY_OPTIONS: Record<
<br />
with exit code 0). Currently, if the software is uninstalled, the
<br />
&quot;installed&quot; status won&apos;t be updated.
&quot;Installed&quot; status won&apos;t be updated.
</>
),
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -111,16 +111,19 @@ const AddPackage = ({
) {
renderFlash(
"error",
`${reason}. ${(
<>
{reason}{" "}
<CustomLink
newTab
url={`${LEARN_MORE_ABOUT_BASE_LINK}/read-package-version`}
text="Learn more"
iconColor="core-fleet-white"
/>
)} `
</>
);
} else {
renderFlash("error", getErrorMessage(e));
}
renderFlash("error", getErrorMessage(e));
}

onExit();
Expand Down

0 comments on commit f135d5b

Please sign in to comment.