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

issue-1, issue-2, Shotover blog has been created #4

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

GrandShow
Copy link

@netlify
Copy link

netlify bot commented Jan 19, 2023

Deploy Preview for infallible-bartik-4f03f3 ready!

Name Link
🔨 Latest commit 43b73d3
🔍 Latest deploy log https://app.netlify.com/sites/infallible-bartik-4f03f3/deploys/63c92f368a2ed4000a8d9891
😎 Deploy Preview https://deploy-preview-4--infallible-bartik-4f03f3.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 settings.

@benbromhead
Copy link
Member

Thanks @GrandShow, I'll look to give you access to netlify to sort out the change in build process / publishing

@benbromhead
Copy link
Member

A few comments, mainly to simplify things and clean it up. Otherwise it looks great!

  • image

Appears to have a blue bar underneath the image. I think it's trying to border the picture, but it ends up looking like css style gone wrong. Maybe if the blue bar ended where the blue twist (on the right side of it did).

  • Let's get rid of the text down the bottom:

image

  • Let's simplify some of the landing page a little bit and get rid of the roadmap part and deploying shotover.
  • Blog landing page. Let's get rid of the preview images, just the text preview. Change "Welcome to our blog" to "Shotover Proxy blog"
  • Get rid of the "Contacts" section.

@taaraora
Copy link
Contributor

taaraora commented Feb 3, 2023

@benbromhead thank you, I'll fix this.

A few comments, mainly to simplify things and clean it up. Otherwise it looks great!

  • image

Appears to have a blue bar underneath the image. I think it's trying to border the picture, but it ends up looking like css style gone wrong. Maybe if the blue bar ended where the blue twist (on the right side of it did).

  • Let's get rid of the text down the bottom:
image
  • Let's simplify some of the landing page a little bit and get rid of the roadmap part and deploying shotover.
  • Blog landing page. Let's get rid of the preview images, just the text preview. Change "Welcome to our blog" to "Shotover Proxy blog"
  • Get rid of the "Contacts" section.

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.

Add blog to website
3 participants