Skip to content
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

Gateway General Settings Page #763

Merged
merged 4 commits into from
May 28, 2019
Merged

Conversation

bafonins
Copy link
Contributor

Summary

Closes #749
This PR adds the Gateway General Settings Page

Changes

  • Add <GatewayDataForm />
  • Add <GatewayGeneralSettigns />
  • Refactor <GatewayAdd />

Notes for Reviewers

Is it fine to put the form to /console/components or is it more suitable for containers ?

Release Notes

  • Added Gateway General Settings Page to the console

@bafonins bafonins added the c/console This is related to the Console label May 27, 2019
@bafonins bafonins added this to the May 2019 milestone May 27, 2019
@bafonins bafonins requested a review from kschiffer May 27, 2019 14:36
@bafonins bafonins self-assigned this May 27, 2019
@coveralls
Copy link

coveralls commented May 27, 2019

Coverage Status

Coverage decreased (-0.08%) to 73.05% when pulling 341cab2 on feature/749-gateway-general-settings into d42b564 on master.

Copy link
Contributor

@kschiffer kschiffer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only some small things.

error: '',
initialValues: {},
mapErrorsToFields: {},
formRef: undefined,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
formRef: undefined,

?

},
dispatch => ({
onDeleteSuccess: () => dispatch(replace('/console/gateways')),
updateGateway: (id, patch) => dispatch(updateGateway(id, patch)),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
updateGateway: (id, patch) => dispatch(updateGateway(id, patch)),
updateGateway,

<Row>
<Col lg={8} md={12}>
<IntlHelmet
title={sharedMessages.addGateway}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
title={sharedMessages.addGateway}
title={sharedMessages.generalSettings}

@bafonins bafonins requested a review from kschiffer May 28, 2019 08:55
@bafonins bafonins force-pushed the feature/749-gateway-general-settings branch from 1adffa6 to 341cab2 Compare May 28, 2019 08:57
@bafonins bafonins requested review from kschiffer and removed request for kschiffer May 28, 2019 08:58
@bafonins bafonins merged commit 62bef38 into master May 28, 2019
@bafonins bafonins deleted the feature/749-gateway-general-settings branch May 28, 2019 09:17
@kschiffer kschiffer mentioned this pull request May 28, 2019
48 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/console This is related to the Console
Projects
None yet
Development

Successfully merging this pull request may close these issues.

General Settings view for Gateways
3 participants