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.

Sign up for GitHub

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

Jump to bottom

Updates to partner page design to add KCSPs #5336

Merged
merged 8 commits into from
Sep 11, 2017
Merged

Updates to partner page design to add KCSPs #5336

merged 8 commits into from
Sep 11, 2017

Conversation

kbarnard10
Copy link
Contributor

@kbarnard10 kbarnard10 commented Sep 7, 2017 •
edited by k8s-reviewable
Loading

Do not merge until 9/11 at 9am PT

Final KCSPs still need to be confirmed (logos are currently placeholders), but page layout is finalized and ready to test otherwise.

@bobsky @zacharysarah

This change is Reviewable

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Sep 7, 2017
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Sep 7, 2017 •
edited
Loading

Deploy preview ready!

Built with commit 48fd766

https://deploy-preview-5336--kubernetes-io-master-staging.netlify.com

@chenopis chenopis added do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. Docs LGTM labels Sep 7, 2017
@dankohn
Copy link
Contributor

dankohn commented Sep 8, 2017

Could we please switch to the Bitnami and Heptio logos we use on https://www.cncf.io/about/members/ ? I think including the company name in the logo makes it much more visually clear who we're talking about, even if you lose a bit of the design aesthetic.

Also, Skippbox needs to be removed from the Service Providers section (they were acquired by Bitnami) and StackPointCloud needs to be removed (since they're in the KCSP section).

Thanks.

@dankohn
Copy link
Contributor

dankohn commented Sep 8, 2017 •
edited
Loading

Also, please remove Deis from Service Providers, as they were acquired by Microsoft.

@kbarnard10
Copy link
Contributor Author

@dankohn I made your requested updates. Also added Booz Allen Hamilton.

@dankohn
Copy link
Contributor

dankohn commented Sep 9, 2017

Could you please remove Deis from this version or in a new PR?

@kbarnard10
Copy link
Contributor Author

@dankohn Yup! Done - I had missed their second listing.

@kbarnard10
Copy link
Contributor Author

@zacharysarah Can you merge this now?

@zacharysarah zacharysarah merged commit a9208e8 into kubernetes:master Sep 11, 2017
@chenopis chenopis removed the do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. label Sep 14, 2017
jesscodez pushed a commit that referenced this pull request Sep 22, 2017
@kbarnard10
Updates to partner page design to add KCSPs (#5336)
dc3fdd7 
* Updates to partner page design to add KCSPs

* Update index.html

* Update partner-style.css

* Update partner-script.js

* Add files via upload

* Update partner-script.js

* Add files via upload

* Update partner-script.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zacharysarah zacharysarah Awaiting requested review from zacharysarah

Assignees

@chenopis chenopis

Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@kbarnard10 @k8sio-netlify-preview-bot @dankohn @zacharysarah @k8s-ci-robot @chenopis