Skip to content

Add Sift Security logo with link to blog post.#43

Open
mbrukman wants to merge 1 commit into
JanusGraph:masterfrom
mbrukman:add-siftsecurity-logo
Open

Add Sift Security logo with link to blog post.#43
mbrukman wants to merge 1 commit into
JanusGraph:masterfrom
mbrukman:add-siftsecurity-logo

Conversation

@mbrukman

Copy link
Copy Markdown
Member

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

@rcanzanese, please review.

@hsaputra hsaputra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@mbrukman

mbrukman commented Jun 4, 2018

Copy link
Copy Markdown
Member Author

@hsaputra, thanks for the review!

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

Signed-off-by: Misha Brukman <mbrukman@google.com>
@mbrukman mbrukman force-pushed the add-siftsecurity-logo branch from f87ab48 to c682aab Compare July 31, 2018 22:59
@mbrukman

Copy link
Copy Markdown
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

porunov commented May 16, 2019

Copy link
Copy Markdown
Member

@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

rcanzanese commented Jun 4, 2019

Copy link
Copy Markdown

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

porunov commented Jul 7, 2019

Copy link
Copy Markdown
Member

@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
Copy Markdown
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