Skip to content

Commit

Permalink
console: Fix desired rx1 settings
Browse files Browse the repository at this point in the history
  • Loading branch information
kschiffer authored and adriansmares committed Feb 14, 2023
1 parent b14ae59 commit b62fc3e
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ For details about compatibility between different releases, see the **Commitment
### Fixed

- Key unwrap caching.
- Desired RX1 delay and desired beacon frequency not being possible to set for OTAA devices.

### Security

Expand Down
2 changes: 1 addition & 1 deletion pkg/webui/console/components/mac-settings-section/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ const MacSettingsSection = props => {
const [isCollapsed, setIsCollapsed] = React.useState(initiallyCollapsed)
const handleIsCollapsedChange = React.useCallback(() => {
if (!isCollapsed && pingPeriodicityRequired) {
// Do not close section if `ping_slot_perdiodicity` is required.
// Do not close section if `ping_slot_periodicity` is required.
return
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ const validationSchema = Yup.object()
)
}),
desired_beacon_frequency: Yup.lazy(() => {
if (!isClassB || mode !== ACTIVATION_MODES.MULTICAST) {
if (!isClassB || mode === ACTIVATION_MODES.MULTICAST) {
return Yup.number().strip()
}

Expand Down Expand Up @@ -180,7 +180,7 @@ const validationSchema = Yup.object()
.max(15, Yup.passValues(sharedMessages.validateNumberLte))
}),
desired_rx1_delay: Yup.lazy(delay => {
if (delay === undefined || delay === '' || mode !== ACTIVATION_MODES.MULTICAST) {
if (delay === undefined || delay === '' || mode === ACTIVATION_MODES.MULTICAST) {
return Yup.number().strip()
}

Expand Down

0 comments on commit b62fc3e

Please sign in to comment.