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

Add doc for firewall auto-detection #307

Merged
merged 1 commit into from
Oct 26, 2016
Merged

Add doc for firewall auto-detection #307

merged 1 commit into from
Oct 26, 2016

Conversation

mdlinville
Copy link

Describe the proposed changes

Added docs about firewall auto-detection on D4W

Project version

Beta 29

Related issue

n/a

Related issue or PR in another project

https://github.com/docker/pinata/issues/5593

Please take a look

@simonferquel @londoncalling

@mdlinville
Copy link
Author

mdlinville commented Oct 21, 2016

@simonferquel
Copy link
Contributor

The popup will look a bit more like this:
image

Signed-off-by: Misty Stanley-Jones <misty@docker.com>
@mdlinville
Copy link
Author

I updated the image with the one you provided, @simonferquel

@simonferquel
Copy link
Contributor

Lgtm

@mdlinville mdlinville modified the milestone: pinata/beta29 Oct 25, 2016
@mdlinville
Copy link
Author

Waiting until beta29 release to merge.

@mdlinville
Copy link
Author

I took the screenshot from Windows. Has the artwork changed since I took
the screenshot? Is this one final enough that I can use it?

On Oct 24, 2016, at 1:58 AM, Simon Ferquel notifications@github.com wrote:

The popup will look a bit more like this:
[image: image]
https://cloud.githubusercontent.com/assets/1677333/19639426/b93c9f24-99d8-11e6-86e2-6a989d08794c.png


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#307 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHUa9UqnsPmJkqmiHoiBnHg4xUFgThbRks5q3HMggaJpZM4KdaNN
.

@mdlinville mdlinville merged commit cdbebff into docker:master Oct 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants