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

test(pie-docs): 404 test fix #307

Merged
merged 11 commits into from
Mar 13, 2023
Merged

test(pie-docs): 404 test fix #307

merged 11 commits into from
Mar 13, 2023

Conversation

siggerzz
Copy link
Contributor

@siggerzz siggerzz commented Mar 13, 2023

Describe your changes (can list changeset entries if preferable)

Author Checklist (complete before requesting a review)

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • If it is a PIE Docs change, I have reviewed the PR preview
  • If there are visual test updates, I have reviewed them properly before approving

Reviewer checklists (complete before approving)

Reviewer 1

  • If it is a PIE Docs change, I have reviewed the PR preview
  • If there are visual test updates, I have reviewed them

Reviewer 2

  • If it is a PIE Docs change, I have reviewed the PR preview
  • If there are visual test updates, I have reviewed them

@changeset-bot
Copy link

changeset-bot bot commented Mar 13, 2023

⚠️ No Changeset found

Latest commit: 47d989e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Mar 13, 2023

PR Preview Action v1.3.0
Preview removed because the pull request was closed.
2023-03-13 11:46 UTC

@siggerzz siggerzz merged commit 7bf4289 into main Mar 13, 2023
@siggerzz siggerzz deleted the 404-test-fix branch March 13, 2023 11:45
FayeCarter pushed a commit that referenced this pull request Mar 15, 2023
* test(pie-monorepo): added Playwright and new system tests to Pie docs

* ci(pie-monorepo): added npx playwright install

* ci(pie-monorepo): updating playwright config for CI

* test(pie-docs): update test cookie banner cookies

* ci(pie-monorepo): updated build.yml playwright install step

* ci(pie-docs): updated cookies domain and path

* ci(pie-monorepo): updated playwright install in CI steps

* test(pie-docs): fix 404 test

---------

Co-authored-by: Joshua Ng <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>
FayeCarter added a commit that referenced this pull request Mar 15, 2023
* feat(wc-react18): add pie button to react

* feat(wc-react18): add pie-button and styles

* docs(pie-docs): content added and amended for the Engineers section of the docs site (#298)

* cosmetic(pie-docs): use link cards for storybook links (#299)

* cosmetic(pie-docs): use link cards for storybook links

* fix(pie-docs): use valid aria-hidden attribute values in footer dividers

* chore(pie-docs): add changeset file

* docs(pie-docs): tweak copy in our-vision page

* cosmetic(pie-docs): use higher quality images for storybook cards

* cosmetic(pie-docs): add an hr below the new cards

* cosmetic(pie-docs): limit label border to image cards

* cosmetic(pie-docs): add captioned image to engineers contributing page

* chore(pie-docs): add changeset file

* refactor(pie-docs): combine hasImage styling under one class modifier for cards

* refactor(pie-docs): move card border styling to scss variable

* fix(pie-docs): use type/ttf for ptmono preloading type

* Version Packages (#296)

release: release packages with 'latest' tag

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* docs(pie-docs): carry out link audit on the docs site (#300)

* docs(pie-docs): carry out link audit on the docs site

* docs(pie-docs): [Fixed] Small changes from self-review

* test(pie-monorepo): added Playwright and new system tests to Pie docs (#294)

* test(pie-monorepo): added Playwright and new system tests to Pie docs

* ci(pie-monorepo): added npx playwright install

* ci(pie-monorepo): updating playwright config for CI

* test(pie-docs): update test cookie banner cookies

* ci(pie-monorepo): updated build.yml playwright install step

* ci(pie-docs): updated cookies domain and path

* ci(pie-monorepo): updated playwright install in CI steps

* test(pie-docs): updated page routing for tests

* ci(pie-monorepo): added reporter upload for CI

* test(pie-docs): updated pathName

* test(pie-docs): updated baseUrl

* ci(pie-monorepo): cache playwright deps

* test(pie-monorepo): increase parallel workers to match cpu cores

* ci(pie-monorepo): ensure playwright deps get installed after cache restore

* ci(pie-monorepo): fix indentation

* ci(pie-monorepo): fix indentation v2

* ci(pie-monorepo): fix indentation v3

* test(pie-monorepo): modify test config to reduce duplicate package.json scripts

* chore(pie-monorepo): remove dependency

* chore(pie-monorepo): update turborepo to allow for passing of root-level args

* chore(pie-monorepo): add changeset

* fix(pie-monorepo): specify outputs in turbo.json

* test(pie-docs): simplify cookie banner tests

* format(pie-docs): add comment consistency in tests

* format(pie-docs): fix linting

* refactor(pie-docs): refactored refresh page in test and addressed comments

---------

Co-authored-by: Joshua Ng <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>

* docs(pie-docs): removed line break on 404 page (#289)

* docs(pie-docs): removed line break on 404 page

* docs(pie-docs): set max-width for centred page text

* test(pie-docs): add missing a11y and system tests for docs site 404 page

* test(pie-docs): [Fixed] Compare paths for 404-to-homepage test rather than full URL

* fix(pie-docs): add missing data-test-id in 404 page

* test(pie-docs): refactor test to use playwright

* fix(pie-docs): fix test in CI

---------

Co-authored-by: Ben Siggery <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>

* test(pie-docs): 404 test fix (#307)

* test(pie-monorepo): added Playwright and new system tests to Pie docs

* ci(pie-monorepo): added npx playwright install

* ci(pie-monorepo): updating playwright config for CI

* test(pie-docs): update test cookie banner cookies

* ci(pie-monorepo): updated build.yml playwright install step

* ci(pie-docs): updated cookies domain and path

* ci(pie-monorepo): updated playwright install in CI steps

* test(pie-docs): fix 404 test

---------

Co-authored-by: Joshua Ng <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>

* test(pie-docs): system tests - footer links (#297)

* test(pie-docs): new test for footer privacy policy

* format(pie-docs): fix linting

* test(pie-monorepo): added Playwright and new system tests to Pie docs

* ci(pie-monorepo): added npx playwright install

* ci(pie-monorepo): updating playwright config for CI

* test(pie-docs): update test cookie banner cookies

* ci(pie-monorepo): updated build.yml playwright install step

* ci(pie-docs): updated cookies domain and path

* format(pie-docs): correct variable name

* ci(pie-monorepo): updated playwright install in CI steps

* test(pie-docs): updated page routing for tests

* ci(pie-monorepo): added reporter upload for CI

* test(pie-docs): updated pathName

* test(pie-docs): updated baseUrl

* ci(pie-monorepo): cache playwright deps

* test(pie-monorepo): increase parallel workers to match cpu cores

* ci(pie-monorepo): ensure playwright deps get installed after cache restore

* ci(pie-monorepo): fix indentation

* ci(pie-monorepo): fix indentation v2

* ci(pie-monorepo): fix indentation v3

* test(pie-monorepo): modify test config to reduce duplicate package.json scripts

* chore(pie-monorepo): remove dependency

* chore(pie-monorepo): update turborepo to allow for passing of root-level args

* chore(pie-monorepo): add changeset

* fix(pie-monorepo): specify outputs in turbo.json

* test(pie-docs): simplify cookie banner tests

* format(pie-docs): add comment consistency in tests

* format(pie-docs): fix linting

* test(pie-docs): refactor footer tests to playwright

* chore(pie-docs): resolve conflict

* chore(pie-docs): solve conflicts

* refactor(pie-docs): refactor test

---------

Co-authored-by: Ben Siggery <[email protected]>
Co-authored-by: Joshua Ng <[email protected]>
Co-authored-by: Josh Ng <[email protected]>

* test(pie-docs): refactor accessibility tests to use playwright (#302)

* test(pie-monorepo): added Playwright and new system tests to Pie docs

* ci(pie-monorepo): added npx playwright install

* ci(pie-monorepo): updating playwright config for CI

* test(pie-docs): update test cookie banner cookies

* ci(pie-monorepo): updated build.yml playwright install step

* ci(pie-docs): updated cookies domain and path

* ci(pie-monorepo): updated playwright install in CI steps

* test(pie-docs): refactor accessibility tests to use playwright

* format(pie-docs): improve test format

* chore(pie-docs): pin deps

* chore(pie-docs): update import

* format(pie-docs): destructure baseURL to match other tests

---------

Co-authored-by: Joshua Ng <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>

* feat(wc-react18): add custom event

* Version Packages (#301)

release: release packages with 'latest' tag

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* feat(wc-vue3) - added empty vue 3 app (#311)

* feat(wc-vue3) - added empty vue 3 app

chore(wc-vue3) - simplified setup
chore(wc-vue3) - updated yarn.lock
chore(wc-vue3) - added eslint config

* chore(wc-vue3) - changeset

* chore(wc-vue3) - remove unnecessary vue plugin

* test(pie-docs): updated visual tests to support playwright (#312)

* test(pie-monorepo): added Playwright and new system tests to Pie docs

* test(pie-docs): update test cookie banner cookies

* ci(pie-docs): updated cookies domain and path

* test(pie-docs): refactor accessibility tests to use playwright

* test(pie-docs): added percy playwright integration

* wip: add custom fixture

* refactor(pie-docs): ensure visual tests use playwright

* test(pie-docs): updated visual tests to support playwright

* fix(pie-docs): re-added babel config

* refactor(pie-docs): remove comment from babel config

* performance(pie-docs): updated playwright workers

* chore(pie-monorepo): rebase

* chore(pie-monorepo): removed already versioned changesets

* chore(pie-monorepo): addressed PR comments

* chore(pie-monorepo): addressed PR comments

* chore(pie-docs): grammar

---------

Co-authored-by: Joshua Ng <[email protected]>
Co-authored-by: siggerzz <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>

* docs(pie-docs): remove our vision engineers page and update roadmap (#315)

* docs(pie-docs): remove our vision engineers page and update roadmap

* test(pie-docs): remove test route

* docs(pie-docs): updated support section copy (#316)

* docs(pie-docs): update home page copy (#317)

* docs(pie-docs): copy updates to what is pie and our vision (#318)

* docs(pie-docs): copy updates to what is pie and our vision

* cosmetic(pie-docs): add ? to What is PIE nav key

* feat(wc-react18): remove ^

* feat(wc-react18): add changesets

* feat(wc-react18): update readme

* Update README.md

* Update README.md

* chore(wc-react18): add custom eslint settings enabling React parsing

* feat(wc-react18): update from review comments

* fix(wc-react18): load fonts using a link tag

* chore(wc-react18): use dev to start server

* chore(wc-react18): add volta settings to package.json

---------

Co-authored-by: Ashley Watson-Nolan <[email protected]>
Co-authored-by: Jamie Maguire <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Xander Marjoram <[email protected]>
Co-authored-by: Josh Ng <[email protected]>
Co-authored-by: Joshua Ng <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>
Co-authored-by: Ben Siggery <[email protected]>
Co-authored-by: Fernando de França <[email protected]>
Co-authored-by: Raouf <[email protected]>
Co-authored-by: Raouf <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants