Skip to content

Commit

Permalink
Don't crash if values are empty (#882)
Browse files Browse the repository at this point in the history
  • Loading branch information
laszlocph authored Nov 16, 2024
1 parent e136dc8 commit e61a643
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions web/src/views/envConfig/envConfig.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ export function EnvConfig(props) {
}, [selectedTemplate, stackConfigDerivedValues]);

useEffect(() => {
if (configFile && configFile.values.ingress) {
if (configFile && configFile.values && configFile.values.ingress) {
if (configFile.values.ingress.protectWithOauthProxy && stackConfigDerivedValues) {
setConfigFile(prevState => ({
...prevState,
Expand All @@ -131,7 +131,7 @@ export function EnvConfig(props) {
}
}
// eslint-disable-next-line react-hooks/exhaustive-deps
}, [configFile && configFile.values.ingress && configFile.values.ingress.protectWithOauthProxy, stackConfigDerivedValues])
}, [configFile && configFile.values && configFile.values.ingress && configFile.values.ingress.protectWithOauthProxy, stackConfigDerivedValues])

const setAppName = (appName) => {
setConfigFile(prevState => ({
Expand Down

0 comments on commit e61a643

Please sign in to comment.