Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Added links section to network container tab#2556

Merged
FrenchBen merged 3 commits intodocker-archive-public:masterfrom
daaru00:container-links
Jun 13, 2017
Merged

Added links section to network container tab#2556
FrenchBen merged 3 commits intodocker-archive-public:masterfrom
daaru00:container-links

Conversation

@daaru00
Copy link
Copy Markdown
Contributor

@daaru00 daaru00 commented Apr 9, 2017

List linked containers and open container page.
Allow to manage container links adding new or remove an existing existing one.
Add style to select input.
links_restyle

- list linked containers
- add and remove links
- restyle select input
- added new link validation

Signed-off-by: Fabio Gollinucci <fbgolly@gmail.com>
@FrenchBen
Copy link
Copy Markdown
Contributor

@daaru00 you're moving fast on the PRs 💟 👍

Unfortunately links are somewhat of a legacy concept; could we perhaps show the list of containers on the same network instead?

daaru00 added 2 commits April 16, 2017 10:03
Signed-off-by: Fabio Gollinucci <fbgolly@gmail.com>
Signed-off-by: Fabio Gollinucci <fbgolly@gmail.com>
@daaru00
Copy link
Copy Markdown
Contributor Author

daaru00 commented Apr 20, 2017

With latest commit when container start all linked container will be automatic start (if the new option in preference is set)
preference-linked

Copy link
Copy Markdown
Contributor

@FrenchBen FrenchBen left a comment

Choose a reason for hiding this comment

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

LGTM

@FrenchBen FrenchBen merged commit 0e5842f into docker-archive-public:master Jun 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants