New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(xo-web/backup-ng): display user errors in the form #4131
Conversation
: setting | ||
) | ||
return mapValues(settings, predicate) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inline predicate.
@@ -134,11 +136,13 @@ const getOptionRenderer = ({ label }) => <span>{_(label)}</span> | |||
|
|||
const createDoesRetentionExist = name => { | |||
const predicate = setting => setting[name] > 0 | |||
return ({ propSettings, settings = propSettings }) => settings.some(predicate) | |||
return ({ settings }) => some(settings, predicate) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can inline predicate.
Why you use _.some
instead of some
? settings
is an Array ?
newSchedule.name !== oldSchedule.name || | ||
newSchedule.timezone !== oldSchedule.timezone || | ||
newSchedule.enabled !== oldSchedule.enabled | ||
) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you remove this ? it avoid editSchedule
if shedule
doesn't change, right?
6ade22c
to
318f9b3
Compare
318f9b3
to
46802d3
Compare
46802d3
to
c520f2b
Compare
f0dcc88
to
d3a5ae3
Compare
Fixes #3831
Check list
Fixes #007
)CHANGELOG.unreleased.md
:${name} v${new version}
)Process
WiP:
(Work in Progress) if not ready to be merged