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

Remove Double CTA on homepage #7322

Merged
merged 2 commits into from
Oct 21, 2024
Merged

Remove Double CTA on homepage #7322

merged 2 commits into from
Oct 21, 2024

Conversation

kukicado
Copy link
Contributor

Before you merge this PR, please review the following:

Blog post

  • Checked title case (This is correct. This Is Not Correct)
  • Updated publishDate
  • Updated authors
  • Updated and checked slug
  • Proofread for spelling and grammar errors
  • Checked for consistent tone and voice throughout via Grammarly
  • Ensured all links are working correctly
  • Added relevant tags and categories
  • Optimized meta description
  • Included a new hero and social/OG image
  • Uploaded images to Google Storage
  • Formatted text for readability (headings, bullet points, etc.)
  • Added image alt text
  • Ran Markdown linter

Other

  • This is not a blog post

* Removed unused `classNames` import
* Removed the homepage CTAs section as it is no longer needed
* Removed unused `classNames` import
* Removed the homepage CTAs section as it is no longer needed
Copy link

netlify bot commented Oct 21, 2024

Deploy Preview for sourcegraph ready!

Name Link
🔨 Latest commit 1fb4b99
🔍 Latest deploy log https://app.netlify.com/sites/sourcegraph/deploys/6715b7365cdbf40008c2b34f
😎 Deploy Preview https://deploy-preview-7322--sourcegraph.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@kukicado kukicado merged commit 893056f into main Oct 21, 2024
6 checks passed
@kukicado kukicado deleted the double-cta-remove branch October 21, 2024 02:11
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.

1 participant