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 SoFi to carousel and fix various things #381

Merged
merged 4 commits into from Nov 22, 2019
Merged

Conversation

@vanesa
Copy link
Member

vanesa commented Nov 22, 2019

Add SoFi to carrousel

image

And also:

  • update the SoFi preview image on blog page to have the correct SoFi logo
  • update convoy logo to svgs
  • move SoFi logo to external logos folder
  • fix some prettier issues with the Quantcast case study
  • diminish size of logos on the case study pages
  • add Convoy PDFs
vanesa added 2 commits Nov 22, 2019
@vanesa vanesa requested review from ryan-blunden and dadlerj Nov 22, 2019
customer: 'SoFi',
logo: '/external-logos/sofi-logo-white.png',
quote:
'Sourcegraph is an ingrained part of my daily process at SoFi. With Sourcegraph, our engineers can understand all of the repercussions of committing a change to a service that is exposed to other services.',

This comment has been minimized.

Copy link
@dadlerj

dadlerj Nov 22, 2019

Member

I'm torn — this is a long quote, and I'm wondering if it would be more impactful with just the 2nd sentence (focused on the use case) or the first (the personal vouch) or sticking with both and just leaving it really long...

This comment has been minimized.

Copy link
@vanesa

vanesa Nov 22, 2019

Author Member

Lets just take the second half.

Co-Authored-By: Ryan Blunden <ryan@sourcegraph.com>
@vanesa vanesa merged commit 3572c2e into master Nov 22, 2019
5 checks passed
5 checks passed
Header rules No header rules processed
Details
Pages changed 708 new files uploaded
Details
Mixed content No mixed content detected
Details
Redirect rules 78 redirect rules processed
Details
deploy/netlify Deploy preview ready!
Details
@vanesa vanesa deleted the vo/add-SoFi-to-carousel branch Nov 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.