Skip to content

Commit

Permalink
console: Fix PR suggestions
Browse files Browse the repository at this point in the history
  • Loading branch information
@PavelJankoski
PavelJankoski committed Nov 21, 2023
1 parent 5c157ce commit f84566c
Show file tree
Hide file tree
Showing 3 changed files with 107 additions and 155 deletions.
22 changes: 9 additions & 13 deletions pkg/webui/console.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ import sentryConfig from '@ttn-lw/constants/sentry'
import { BreadcrumbsProvider } from '@ttn-lw/components/breadcrumbs/context'
import Header from '@ttn-lw/components/header'

import { EnvProvider } from '@ttn-lw/lib/components/env'
import { ErrorView } from '@ttn-lw/lib/components/error-view'
import { FullViewError } from '@ttn-lw/lib/components/full-view-error'
import Init from '@ttn-lw/lib/components/init'
Expand All @@ -33,7 +32,6 @@ import Logo from '@console/containers/logo'

import App from '@console/views/app'

import env from '@ttn-lw/lib/env'
import { selectApplicationRootPath, selectSentryDsnConfig } from '@ttn-lw/lib/selectors/env'

import createStore from './console/store'
Expand All @@ -58,17 +56,15 @@ const errorRender = error => (

DOM.render(
<ErrorView errorRender={errorRender}>
<EnvProvider env={env}>
<Provider store={store}>
<WithLocale>
<Init>
<BreadcrumbsProvider>
<App history={history} />
</BreadcrumbsProvider>
</Init>
</WithLocale>
</Provider>
</EnvProvider>
<Provider store={store}>
<WithLocale>
<Init>
<BreadcrumbsProvider>
<App history={history} />
</BreadcrumbsProvider>
</Init>
</WithLocale>
</Provider>
</ErrorView>,
rootElement,
)
197 changes: 98 additions & 99 deletions pkg/webui/console/components/location-form/index.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -145,15 +145,16 @@ const LocationForm = props => {
setLatitude(latitude)
setLongitude(longitude)
setMapCenter([latitude, longitude])
setLoading(false)
},
() => {
setMapCenter(defaultLocation)
setZoom(2)
setLoading(false)
},
)
}
}
setLoading(false)
}, [additionalMarkers.length, initialValues])

const handleSubmit = useCallback(
Expand Down Expand Up @@ -245,106 +246,104 @@ const LocationForm = props => {
}, [deleteAll, entityId, onDelete])

return (
<React.Fragment>
<Form
error={error}
validateOnChange
initialValues={initialValues}
validationSchema={validationSchema}
onSubmit={handleSubmit}
formikRef={form}
>
<Form.SubTitle title={formTitle} />
{children}
<Message content={sharedMessages.location} component="h4" className="mb-cs-xs mt-0" />
{!entryExists && <Notification content={noLocationSetInfo} info small />}
<Overlay loading={loading} visible={loading} spinnerMessage={m.loadingLocation}>
<LocationMap
widget
leafletConfig={{ zoom, minZoom: 1 }}
mapCenter={mapCenter}
markers={markers}
onClick={handleClick}
clickable
centerOnMarkers
/>
</Overlay>
<Message
content={updatesDisabled ? m.mapDescriptionDisabled : m.mapDescription}
component="p"
className="p-0 mt-cs-xs mb-cs-l tc-subtle-gray"
<Form
error={error}
validateOnChange
initialValues={initialValues}
validationSchema={validationSchema}
onSubmit={handleSubmit}
formikRef={form}
>
<Form.SubTitle title={formTitle} />
{children}
<Message content={sharedMessages.location} component="h4" className="mb-cs-xs mt-0" />
{!entryExists && <Notification content={noLocationSetInfo} info small />}
<Overlay loading={loading} visible={loading} spinnerMessage={m.loadingLocation}>
<LocationMap
widget
leafletConfig={{ zoom, minZoom: 1 }}
mapCenter={mapCenter}
markers={markers}
onClick={handleClick}
clickable
centerOnMarkers
/>
{updatesDisabled && disabledInfo && <Notification content={disabledInfo} info small />}
<Form.Field
type="number"
step="any"
title={sharedMessages.latitude}
description={sharedMessages.latitudeDesc}
name="latitude"
component={Input}
required={!updatesDisabled}
disabled={updatesDisabled}
onBlur={handleLatitudeChange}
</Overlay>
<Message
content={updatesDisabled ? m.mapDescriptionDisabled : m.mapDescription}
component="p"
className="p-0 mt-cs-xs mb-cs-l tc-subtle-gray"
/>
{updatesDisabled && disabledInfo && <Notification content={disabledInfo} info small />}
<Form.Field
type="number"
step="any"
title={sharedMessages.latitude}
description={sharedMessages.latitudeDesc}
name="latitude"
component={Input}
required={!updatesDisabled}
disabled={updatesDisabled}
onBlur={handleLatitudeChange}
/>
<Form.Field
type="number"
step="any"
title={sharedMessages.longitude}
description={sharedMessages.longitudeDesc}
name="longitude"
component={Input}
required={!updatesDisabled}
disabled={updatesDisabled}
onBlur={handleLongitudeChange}
/>
<Form.Field
type="number"
step="1"
title={sharedMessages.altitude}
description={sharedMessages.altitudeDesc}
name="altitude"
component={Input}
required={!updatesDisabled}
disabled={updatesDisabled}
/>
<SubmitBar>
<Form.Submit component={SubmitButton} message={sharedMessages.saveChanges} />
<ModalButton
type="button"
icon="delete"
message={m.deleteLocation}
modalData={{
children: (
<div>
<Message
content={onlyAutomaticExists ? m.deleteAllWarning : m.deleteWarning}
component="span"
/>
{entryExists && automaticExists && (
<>
<br />
<br />
<Message content={m.deleteAllInfo} component="span" />
<Checkbox
name="delete-all"
label={m.deleteAllLocations}
onChange={handleDeleteAllCheck}
className="mt-cs-m"
value={deleteAll}
/>
</>
)}
</div>
),
}}
onApprove={handleDelete}
disabled={updatesDisabled || !anyEntryExists}
naked
danger
/>
<Form.Field
type="number"
step="any"
title={sharedMessages.longitude}
description={sharedMessages.longitudeDesc}
name="longitude"
component={Input}
required={!updatesDisabled}
disabled={updatesDisabled}
onBlur={handleLongitudeChange}
/>
<Form.Field
type="number"
step="1"
title={sharedMessages.altitude}
description={sharedMessages.altitudeDesc}
name="altitude"
component={Input}
required={!updatesDisabled}
disabled={updatesDisabled}
/>
<SubmitBar>
<Form.Submit component={SubmitButton} message={sharedMessages.saveChanges} />
<ModalButton
type="button"
icon="delete"
message={m.deleteLocation}
modalData={{
children: (
<div>
<Message
content={onlyAutomaticExists ? m.deleteAllWarning : m.deleteWarning}
component="span"
/>
{entryExists && automaticExists && (
<>
<br />
<br />
<Message content={m.deleteAllInfo} component="span" />
<Checkbox
name="delete-all"
label={m.deleteAllLocations}
onChange={handleDeleteAllCheck}
className="mt-cs-m"
value={deleteAll}
/>
</>
)}
</div>
),
}}
onApprove={handleDelete}
disabled={updatesDisabled || !anyEntryExists}
naked
danger
/>
</SubmitBar>
</Form>
</React.Fragment>
</SubmitBar>
</Form>
)
}

Expand Down
43 changes: 0 additions & 43 deletions pkg/webui/lib/components/env.js
View file

This file was deleted.

0 comments on commit f84566c

Please sign in to comment.