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

Figure redrawn in Google public format #14

Merged
merged 2 commits into from
Aug 24, 2017
Merged

Conversation

mwallman
Copy link
Contributor

@mwallman mwallman commented Aug 7, 2017

No description provided.

@mwallman mwallman requested a review from mdauphinee August 7, 2017 17:33
@woogie-wolgemuth
Copy link

The original image didn't have the header (Architecture: Cloud Connect) or footer (gray bar). Do you think those will look confusing in the context of https://github.com/GoogleCloudPlatform/professional-services/tree/master/infrastructure/cloudconnect ?

@mwallman
Copy link
Contributor Author

mwallman commented Aug 8, 2017

This change came from the SA (Miles) team.

@mwallman mwallman merged commit e9eb75c into master Aug 24, 2017
@ryanmcdowell ryanmcdowell deleted the mwallman-cloudconnect branch November 7, 2018 05:30
morgante pushed a commit that referenced this pull request Mar 11, 2019
# This is the 1st commit message:

functional, permissions need to be modified

# This is the commit message #2:

bug fixes -- passing testing

# This is the commit message #3:

add README

# This is the commit message #4:

fix readme

# This is the commit message #5:

fix readme

# This is the commit message #6:

disable sync interval override

# This is the commit message #7:

add prerequisites to readme

# This is the commit message #8:

add more detail on create.sh compatibility

# This is the commit message #9:

add to create.sh to detect OS

# This is the commit message #10:

fix readme

# This is the commit message #11:

exit create.sh on any errors

# This is the commit message #12:

fix readme

# This is the commit message #13:

fix readme

# This is the commit message #14:

fix readme

# This is the commit message #15:

fix readme

# This is the commit message #16:

deletion works successfully

# This is the commit message #17:

fix readme

# This is the commit message #18:

fix readme

# This is the commit message #19:

fix readme

# This is the commit message #20:

fix readme
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.

3 participants