Skip to content

Commit c447fa2

Browse files
Merge pull request #100 from NessieCanCode/improve-error-message-for-data-loading-failure
Display detailed billing data load errors
2 parents ea8461c + c077ccc commit c447fa2

File tree

2 files changed

+28
-2
lines changed

2 files changed

+28
-2
lines changed

src/slurmcostmanager.css

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,14 @@ nav button:hover {
7979
color: red;
8080
}
8181

82+
.error-details {
83+
border: 1px solid #ccc;
84+
background: #fdd;
85+
padding: 0.5em;
86+
white-space: pre-wrap;
87+
margin-top: 0.5em;
88+
}
89+
8290
.clickable {
8391
cursor: pointer;
8492
}

src/slurmcostmanager.js

Lines changed: 20 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1050,6 +1050,7 @@ function Rates({ onRatesUpdated }) {
10501050
function App() {
10511051
const [view, setView] = useState('summary');
10521052
const { data, error, reload } = useBillingData();
1053+
const [showErrorDetails, setShowErrorDetails] = useState(false);
10531054

10541055
return React.createElement(
10551056
'div',
@@ -1077,9 +1078,26 @@ function App() {
10771078
view !== 'settings' &&
10781079
error &&
10791080
React.createElement(
1080-
'p',
1081+
'div',
10811082
{ className: 'error' },
1082-
`Failed to load data: ${error}`
1083+
React.createElement(
1084+
'p',
1085+
null,
1086+
'Failed to load data ',
1087+
React.createElement(
1088+
'a',
1089+
{
1090+
href: '#',
1091+
onClick: e => {
1092+
e.preventDefault();
1093+
setShowErrorDetails(v => !v);
1094+
}
1095+
},
1096+
'(Click here to Learn More)'
1097+
)
1098+
),
1099+
showErrorDetails &&
1100+
React.createElement('pre', { className: 'error-details' }, error)
10831101
),
10841102
data &&
10851103
view === 'summary' &&

0 commit comments

Comments
 (0)