From 2e2cd1ad963c2800ca811638d95a30811ef101f2 Mon Sep 17 00:00:00 2001 From: Erin Donehoo <105813956+edonehoo@users.noreply.github.com> Date: Fri, 14 Nov 2025 14:06:56 -0500 Subject: [PATCH 1/2] fix: Update links to align with nav redesign. --- .../extensions/data-view/examples/DataView/DataView.md | 2 +- .../content/extensions/data-view/examples/Table/Table.md | 4 ++-- .../content/extensions/data-view/examples/Toolbar/Toolbar.md | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/module/patternfly-docs/content/extensions/data-view/examples/DataView/DataView.md b/packages/module/patternfly-docs/content/extensions/data-view/examples/DataView/DataView.md index 4a3b4fd4..f9f4e9c4 100644 --- a/packages/module/patternfly-docs/content/extensions/data-view/examples/DataView/DataView.md +++ b/packages/module/patternfly-docs/content/extensions/data-view/examples/DataView/DataView.md @@ -30,7 +30,7 @@ The **data view** extension enables you to display datasets in organized layouts ### Layout -A data view should contain a header, the data representation, and a footer. These parts are organized in a [stack layout](/layouts/stack). +A data view should contain a header, the data representation, and a footer. These parts are organized in a [stack layout](/foundations-and-styles/layouts/stack). The data view toolbars and sub-components that display the data (like a card view or table) are always passed as `children` to the `` component. diff --git a/packages/module/patternfly-docs/content/extensions/data-view/examples/Table/Table.md b/packages/module/patternfly-docs/content/extensions/data-view/examples/Table/Table.md index 9ef6a7ce..23a63a83 100644 --- a/packages/module/patternfly-docs/content/extensions/data-view/examples/Table/Table.md +++ b/packages/module/patternfly-docs/content/extensions/data-view/examples/Table/Table.md @@ -149,7 +149,7 @@ When there is a data connection or retrieval error, you can display an error sta The error state will be displayed when the data view `activeState` value is `error`. -You can create your error state by passing either the [component groups extension's error state](/component-groups/error-state) or a [PatternFly empty state](/components/empty-state) to the `error` key of `headStates` or `bodyStates`. +You can create your error state by passing either the [component groups extension's error state](/extensions/component-groups/error-state) or a [PatternFly empty state](/components/empty-state) to the `error` key of `headStates` or `bodyStates`. ```js file="./DataViewTableErrorExample.tsx" @@ -161,7 +161,7 @@ To indicate that data is loading, you can display a loading state. The loading state will be displayed when the data view `activeState` value is `loading`. -You can create your loading state by passing either the [component groups extension's skeleton table](/component-groups/skeleton-table) or a customized [PatternFly empty state](/components/empty-state) to the `loading` key of `headStates` or `bodyStates`. +You can create your loading state by passing either the [component groups extension's skeleton table](/extensions/component-groups/skeleton-table) or a customized [PatternFly empty state](/components/empty-state) to the `loading` key of `headStates` or `bodyStates`. ```js file="./DataViewTableLoadingExample.tsx" diff --git a/packages/module/patternfly-docs/content/extensions/data-view/examples/Toolbar/Toolbar.md b/packages/module/patternfly-docs/content/extensions/data-view/examples/Toolbar/Toolbar.md index d209d228..5daf3a13 100644 --- a/packages/module/patternfly-docs/content/extensions/data-view/examples/Toolbar/Toolbar.md +++ b/packages/module/patternfly-docs/content/extensions/data-view/examples/Toolbar/Toolbar.md @@ -44,7 +44,7 @@ You can further customize toolbar interactions by referring to the additional do ``` ## Toolbar actions -To support additional user needs, you can pass relevant actions to the toolbar via `actions`. Add standard PatternFly actions (like buttons) or choose predefined [responsive actions](/component-groups/responsive-actions) which ensure the responsive behavior of multiple actions in 1 toolbar. +To support additional user needs, you can pass relevant actions to the toolbar via `actions`. Add standard PatternFly actions (like buttons) or choose predefined [responsive actions](/extensions/component-groups/responsive-actions) which ensure the responsive behavior of multiple actions in 1 toolbar. ### Actions example From 57c6beabdfe50a4893e5346febbc3f9400812b7d Mon Sep 17 00:00:00 2001 From: nicolethoen Date: Fri, 14 Nov 2025 16:12:15 -0500 Subject: [PATCH 2/2] fix workflows? --- .github/workflows/build-lint-test.yml | 9 ++++++++- .github/workflows/pr-preview.yml | 13 ++++++++++--- 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/.github/workflows/build-lint-test.yml b/.github/workflows/build-lint-test.yml index e80fe2e0..51eb6f33 100644 --- a/.github/workflows/build-lint-test.yml +++ b/.github/workflows/build-lint-test.yml @@ -155,8 +155,15 @@ jobs: if: steps.dist.outputs.cache-hit != 'true' - name: Build docs run: npm run build:docs + - uses: actions/cache@v4 + id: puppeteer-cache + name: Cache Puppeteer browsers + with: + path: ~/.cache/puppeteer + key: ${{ runner.os }}-puppeteer-${{ hashFiles('package-lock.json') }} - name: Install Chrome for Puppeteer - run: npx puppeteer browsers install chrome + run: npx puppeteer browsers install chrome + if: steps.puppeteer-cache.outputs.cache-hit != 'true' - name: A11y tests run: npm run serve:docs & npm run test:a11y test_component: diff --git a/.github/workflows/pr-preview.yml b/.github/workflows/pr-preview.yml index 86c230c4..463269a3 100644 --- a/.github/workflows/pr-preview.yml +++ b/.github/workflows/pr-preview.yml @@ -34,7 +34,7 @@ jobs: path: '**/node_modules' key: ${{ runner.os }}-npm-14-${{ secrets.CACHE_VERSION }}-${{ hashFiles('package-lock.json') }} - run: npm install --frozen-lockfile - if: steps.yarn-cache.outputs.cache-hit != 'true' + if: steps.npm-cache.outputs.cache-hit != 'true' - run: npm run build name: Build data view - uses: actions/cache@v4 @@ -42,14 +42,21 @@ jobs: name: Load webpack cache with: path: '.cache' - key: ${{ runner.os }}-v4-${{ hashFiles('yarn.lock') }} + key: ${{ runner.os }}-v4-${{ hashFiles('package-lock.json') }} - run: npm run build:docs name: Build docs - run: node .github/upload-preview.js packages/module/public name: Upload docs if: always() + - uses: actions/cache@v4 + id: puppeteer-cache + name: Cache Puppeteer browsers + with: + path: ~/.cache/puppeteer + key: ${{ runner.os }}-puppeteer-${{ hashFiles('package-lock.json') }} - name: Install Chrome for Puppeteer - run: npx puppeteer browsers install chrome + run: npx puppeteer browsers install chrome + if: steps.puppeteer-cache.outputs.cache-hit != 'true' - run: npm run serve:docs & npm run test:a11y name: a11y tests - run: node .github/upload-preview.js packages/module/coverage