-
Notifications
You must be signed in to change notification settings - Fork 20
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@hsaputra, thanks for the review! @rcanzanese, please review and approve when you're ready. Thanks! |
2c06c8a
to
f87ab48
Compare
Signed-off-by: Misha Brukman <[email protected]>
f87ab48
to
c682aab
Compare
@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? |
@mbrukman I know that we are waiting for the approval but now you should rebase your PR from master.
You can see an example of adding new users in the next PRs: #65 #66 |
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. |
@mbrukman @rcanzanese should we include |
5d15717
to
b199941
Compare
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. |
You can see a preview of the resulting page at: https://mbrukman.github.io/janusgraph.org/#users
@rcanzanese, please review.