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

feat(wc-react18): add pie-button to react app #309

Merged
merged 32 commits into from
Mar 15, 2023
Merged

Conversation

FayeCarter
Copy link
Contributor

@FayeCarter FayeCarter commented Mar 14, 2023

Describe your changes (can list changeset entries if preferable)

[Added] pie button to react app
[Updated] button to handle custom events

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 14, 2023

🦋 Changeset detected

Latest commit: 7986d56

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
wc-react18 Minor
@justeattakeaway/pie-button Minor
pie-storybook Patch

Not sure what this means? Click here to learn what changesets are.

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

@github-actions
Copy link
Contributor

github-actions bot commented Mar 14, 2023

PR Preview Action v1.3.0
Preview removed because the pull request was closed.
2023-03-15 14:16 UTC

@FayeCarter FayeCarter marked this pull request as ready for review March 15, 2023 09:08
FayeCarter and others added 18 commits March 15, 2023 10:15
* 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
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

* docs(pie-docs): [Fixed] Small changes from self-review
…#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

* 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-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): 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-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]>
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

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-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]>
…315)

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

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

* cosmetic(pie-docs): add ? to What is PIE nav key
@FayeCarter FayeCarter changed the base branch from main to add-button-component-to-vue-app March 15, 2023 10:17
@FayeCarter FayeCarter changed the base branch from add-button-component-to-vue-app to main March 15, 2023 10:17
@github-actions github-actions bot removed the pie-docs label Mar 15, 2023
@FayeCarter FayeCarter changed the title React pie button feat(wc-react18): add pie-button to react app Mar 15, 2023
Copy link
Contributor

@fernandofranca fernandofranca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a commit for adding custom settings for eslint so it stops complaining about React fragments

apps/examples/wc-react18/public/index.html Outdated Show resolved Hide resolved
packages/components/pie-button/src/index.ts Outdated Show resolved Hide resolved
apps/examples/wc-react18/src/App.js Outdated Show resolved Hide resolved
apps/examples/wc-react18/src/index.js Show resolved Hide resolved
apps/examples/wc-react18/README.md Outdated Show resolved Hide resolved
yarn.lock Outdated Show resolved Hide resolved
Copy link
Contributor

@fernandofranca fernandofranca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉🎉🎉

@FayeCarter FayeCarter merged commit 92a4771 into main Mar 15, 2023
@FayeCarter FayeCarter deleted the react-pie-button branch March 15, 2023 14:12
@github-actions github-actions bot mentioned this pull request Mar 15, 2023
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.

8 participants