newrelic / instant-observability-website

Dashboards, alerts, and integrations all in one place. Our quickstarts bundle everything you need to start monitoring like a pro right out of the box.
https://newrelic.com/instant-observability/
Apache License 2.0
5 stars 20 forks source link

Refactor index page #276

Closed tabathadelane closed 2 years ago

tabathadelane commented 2 years ago

So far we have

Do we need to do more?

mickeyryan42 commented 2 years ago

I also have a branch that moved the category aside and mobile category dropdown into separate components. If thats useful we can look at merging to this branch. I think it simplified the file by a few hundred lines.

gatsby-cloud[bot] commented 2 years ago

Gatsby Cloud Build Report

instant-observability-website

:tada: Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

:clock1: Build time: 1m

mickeyryan42 commented 2 years ago

One consideration is for now i just dropped the new components into the src/components directory. They aren't really shared, so could potentially live somewhere more appropriate.

zstix commented 2 years ago
pages/
  |- index.jsx
components/
  |- foo.jsx
  |- indexComponents/
  |- |- aside.jsx
nr-opensource-bot commented 2 years ago

:tada: This PR is included in version 1.7.1 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: