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

General Settings view for Gateways #749

Closed
bafonins opened this issue May 24, 2019 · 0 comments · Fixed by #763
Closed

General Settings view for Gateways #749

bafonins opened this issue May 24, 2019 · 0 comments · Fixed by #763
Assignees
Labels
c/console This is related to the Console in progress We're working on it
Milestone

Comments

@bafonins
Copy link
Contributor

bafonins commented May 24, 2019

Summary

The console needs to support editing existing gateways.

Why do we need this?

The users of the console should be able to edit gateway related information.

What is already there? What do you see now?

  • updateById and deleteById entries in the gateways service
  • The Gateway Add Page that can be reused.

What is missing? What do you want to see?

  • Entry in the side navigation
  • The page itself

Environment

console (browser)

How do you propose to implement this?

Functionality wise similar to other device/application general settings pages.
UI similar to the Gateway Add Page with disabled Gateway ID field. The rest can be edited by the user.

Screenshot 2019-05-24 at 11 19 00

Can you do this yourself and submit a Pull Request?

yes

cc @htdvisser

@bafonins bafonins added the c/console This is related to the Console label May 24, 2019
@bafonins bafonins added this to the May 2019 milestone May 24, 2019
@bafonins bafonins self-assigned this May 24, 2019
@bafonins bafonins added the in progress We're working on it label May 24, 2019
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 in progress We're working on it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant