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

WIP: Nested nav with reporting on usage #9097

Draft
wants to merge 298 commits into
base: main
Choose a base branch
from
Draft

Conversation

stevector
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Nov 8, 2024

Hi from your friendly robot! 🤖

I've applied Prettier formatting to the following files in fdd46aa:

gatsby-node.js gatsby-ssr.js src/components/omniSidebarNav/all-written-paths.js src/components/omniSidebarNav/checkStatusCodes.js src/components/omniSidebarNav/getOmniItems.js src/components/omniSidebarNav/helpers.js src/components/omniSidebarNav/helpers.test.js src/components/omniSidebarNav/index.js src/components/omniSidebarNav/reporting-run.js src/components/omniSidebarNav/reporting.js src/components/omniSidebarNav/reporting.test.js src/components/omniSidebarNav/submenus/about.js src/components/omniSidebarNav/submenus/accountManagement.js src/components/omniSidebarNav/submenus/certification.js src/components/omniSidebarNav/submenus/frontEndSites.js src/components/omniSidebarNav/submenus/getStarted.js src/components/omniSidebarNav/submenus/goLive.js src/components/omniSidebarNav/submenus/pagesToDelete.js src/components/omniSidebarNav/submenus/security.js src/components/omniSidebarNav/submenus/support.js src/components/omniSidebarNav/submenus/terminus.js src/components/omniSidebarNav/submenus/unassignedPages.js src/components/omniSidebarNav/submenus/webInfrastructure.js src/components/omniSidebarNav/submenus/workflows.js src/components/omniSidebarNav/testfixtures/allPaths.testfixture.js src/layout/header/index.js src/pages/glossary.js src/templates/doc.js src/templates/guide.js src/templates/landing.js src/templates/terminuspage.js src/templates/video.js tests/playwright-tests/visuals.spec.ts

The full diff is below. Please review the changes.

Click to expand
diff --git a/src/templates/doc.js b/src/templates/doc.js
index bf637de5f..7e813d3be 100644
--- a/src/templates/doc.js
+++ b/src/templates/doc.js
@@ -17,7 +17,6 @@ import OmniSidebarNav from '../components/omniSidebarNav';
 const containerWidth = 'standard';
 
 class DocTemplate extends React.Component {
-
   render() {
     const node = this.props.data.doc;
     const isoDate = this.props.data.date;
diff --git a/src/templates/guide.js b/src/templates/guide.js
index 5d086dbac..425cdc7d1 100644
--- a/src/templates/guide.js
+++ b/src/templates/guide.js
@@ -11,8 +11,6 @@ import MdxWrapper from '../components/mdxWrapper';
 import OmniSidebarNav from '../components/omniSidebarNav';
 
 class GuideTemplate extends React.Component {
-
-
   render() {
     const node = this.props.data.mdx;
     const isoDate = this.props.data.date;

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

github-actions bot commented Nov 8, 2024

Hi from your friendly robot! 🤖

I've applied Prettier formatting to the following files in 3237e73:

gatsby-node.js gatsby-ssr.js src/components/omniSidebarNav/all-written-paths.js src/components/omniSidebarNav/checkStatusCodes.js src/components/omniSidebarNav/getOmniItems.js src/components/omniSidebarNav/helpers.js src/components/omniSidebarNav/helpers.test.js src/components/omniSidebarNav/index.js src/components/omniSidebarNav/reporting-run.js src/components/omniSidebarNav/reporting.js src/components/omniSidebarNav/reporting.test.js src/components/omniSidebarNav/submenus/about.js src/components/omniSidebarNav/submenus/accountManagement.js src/components/omniSidebarNav/submenus/certification.js src/components/omniSidebarNav/submenus/frontEndSites.js src/components/omniSidebarNav/submenus/getStarted.js src/components/omniSidebarNav/submenus/goLive.js src/components/omniSidebarNav/submenus/pagesToDelete.js src/components/omniSidebarNav/submenus/security.js src/components/omniSidebarNav/submenus/support.js src/components/omniSidebarNav/submenus/terminus.js src/components/omniSidebarNav/submenus/unassignedPages.js src/components/omniSidebarNav/submenus/webInfrastructure.js src/components/omniSidebarNav/submenus/workflows.js src/components/omniSidebarNav/testfixtures/allPaths.testfixture.js src/layout/header/index.js src/pages/glossary.js src/templates/certificationpage.js src/templates/doc.js src/templates/guide.js src/templates/landing.js src/templates/terminusCommand.js src/templates/terminuspage.js src/templates/video.js tests/playwright-tests/visuals.spec.ts

The full diff is below. Please review the changes.

Click to expand
diff --git a/src/templates/terminusCommand.js b/src/templates/terminusCommand.js
index 8ebbcc646..a2f17aadd 100644
--- a/src/templates/terminusCommand.js
+++ b/src/templates/terminusCommand.js
@@ -11,7 +11,6 @@ import { Container } from '@pantheon-systems/pds-toolkit-react';
 
 let commandsJson = require('../../source/data/commands.json');
 
-
 // @TODO relocate this list
 // - To a YAML file and use GraphQL to pull data.
 // - To a GraphQL query order by frontmatter weight/order/index field.

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

github-actions bot commented Nov 8, 2024

Hi from your friendly robot! 🤖

I've applied Prettier formatting to the following files in 7406d83:

gatsby-node.js gatsby-ssr.js src/components/omniSidebarNav/all-written-paths.js src/components/omniSidebarNav/checkStatusCodes.js src/components/omniSidebarNav/getOmniItems.js src/components/omniSidebarNav/helpers.js src/components/omniSidebarNav/helpers.test.js src/components/omniSidebarNav/index.js src/components/omniSidebarNav/reporting-run.js src/components/omniSidebarNav/reporting.js src/components/omniSidebarNav/reporting.test.js src/components/omniSidebarNav/submenus/about.js src/components/omniSidebarNav/submenus/accountManagement.js src/components/omniSidebarNav/submenus/certification.js src/components/omniSidebarNav/submenus/frontEndSites.js src/components/omniSidebarNav/submenus/getStarted.js src/components/omniSidebarNav/submenus/goLive.js src/components/omniSidebarNav/submenus/pagesToDelete.js src/components/omniSidebarNav/submenus/security.js src/components/omniSidebarNav/submenus/support.js src/components/omniSidebarNav/submenus/terminus.js src/components/omniSidebarNav/submenus/unassignedPages.js src/components/omniSidebarNav/submenus/webInfrastructure.js src/components/omniSidebarNav/submenus/workflows.js src/components/omniSidebarNav/testfixtures/allPaths.testfixture.js src/layout/header/index.js src/pages/glossary.js src/templates/certificationpage.js src/templates/doc.js src/templates/guide.js src/templates/landing.js src/templates/terminusCommand.js src/templates/terminuspage.js src/templates/video.js tests/playwright-tests/visuals.spec.ts

The full diff is below. Please review the changes.

Click to expand
diff --git a/src/components/omniSidebarNav/getOmniItems.js b/src/components/omniSidebarNav/getOmniItems.js
index 8b229798d..62a7185f6 100644
--- a/src/components/omniSidebarNav/getOmniItems.js
+++ b/src/components/omniSidebarNav/getOmniItems.js
@@ -29,8 +29,6 @@ import { simpleLink } from './helpers';
 // Todo: Fix video template (missing search bar)
 // Todo: add sidebar to landing pages (end goal is to delete them entirely)
 
-
-
 /**
  * Retrieves the all the menu items for the sidebar navigation.
  * @returns {Array} An array of Omni items.
diff --git a/src/templates/landing.js b/src/templates/landing.js
index e25a0ad31..bd5458607 100644
--- a/src/templates/landing.js
+++ b/src/templates/landing.js
@@ -11,7 +11,6 @@ import SEO from '../layout/seo';
 // @todo, implement sidebar on this template
 import OmniSidebarNav from '../components/omniSidebarNav';
 
-
 import {
   Container,
   FlexContainer,
@@ -44,13 +43,10 @@ class LandingTemplate extends Component {
         containerWidth={containerWidth}
         footerBorder={topic.footer_border}
       >
-        <SEO title={topic.title}
-          slot="seo"
-        />
+        <SEO title={topic.title} slot="seo" />
         <OmniSidebarNav slot="guide-menu" activePage={topic.path} />
 
-
-        <main id="docs-main" slot="guide-content"  tabIndex="-1">
+        <main id="docs-main" slot="guide-content" tabIndex="-1">
           <Container width={containerWidth} className="landing-page__header">
             <FlexContainer
               alignItems="center"

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9097-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Site: Site Structure Issue or PR with the docs app stack and/or live site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants