Skip to content

Commit 24e89c0

Browse files
committed
Format and reveal billing load errors
1 parent 5fa17ad commit 24e89c0

2 files changed

Lines changed: 34 additions & 2 deletions

File tree

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: 26 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ function useBillingData() {
5151
setError(null);
5252
} catch (e) {
5353
console.error(e);
54-
setError(e);
54+
setError(e.message || String(e));
5555
}
5656
}, []);
5757

@@ -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',
@@ -1074,7 +1075,30 @@ function App() {
10741075
)
10751076
),
10761077
view !== 'settings' && !data && !error && React.createElement('p', null, 'Loading...'),
1077-
view !== 'settings' && error && React.createElement('p', { className: 'error' }, 'Failed to load data'),
1078+
view !== 'settings' &&
1079+
error &&
1080+
React.createElement(
1081+
'div',
1082+
{ className: '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)
1101+
),
10781102
data &&
10791103
view === 'summary' &&
10801104
React.createElement(Summary, {

0 commit comments

Comments
 (0)