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 Sift Security logo with link to blog post. #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mbrukman
Copy link
Member

You can see a preview of the resulting page at: https://mbrukman.github.io/janusgraph.org/#users

@rcanzanese, please review.

Copy link
Member

@hsaputra hsaputra left a comment

Choose a reason for hiding this comment

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

LGTM

@mbrukman
Copy link
Member Author

mbrukman commented Jun 4, 2018

@hsaputra, thanks for the review!

@rcanzanese, please review and approve when you're ready. Thanks!

@mbrukman mbrukman force-pushed the add-siftsecurity-logo branch from f87ab48 to c682aab Compare July 31, 2018 22:59
@mbrukman
Copy link
Member Author

@rcanzanese — with the recent acquisition of Sift Security by Netskope, will Sift Security operate as a separate company/brand or will it be merged into Netskope? Can we still feature Sift Security logo + the JanusGraph use case you published previously?

@porunov
Copy link
Member

porunov commented May 16, 2019

@mbrukman I know that we are waiting for the approval but now you should rebase your PR from master.
To include user into the user section you should do the next:

  1. add logo into img/logos/
  2. add _posts/2019-05-11-{some_name}.markdown file into _posts/.

You can see an example of adding new users in the next PRs: #65 #66

@rcanzanese
Copy link

rcanzanese commented Jun 4, 2019

Apologies for missing this in my github notifications. Sift Security no longer exists as a separate entity. The use case still holds for now. I'll send a message out of band to Misha.

@porunov
Copy link
Member

porunov commented Jul 7, 2019

@mbrukman @rcanzanese should we include Netskope logo in such case or not?

@porunov porunov force-pushed the master branch 2 times, most recently from 5d15717 to b199941 Compare September 4, 2019 15:14
@FlorianHockmann
Copy link
Member

Hey @rcanzanese, wondering whether you could give a quick update on this. Can we add Netskope to the homepage? Otherwise we should simply close this PR.

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.

6 participants