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

Sync canary -> main #1488

Merged
merged 15 commits into from
Jul 11, 2023
Merged

Sync canary -> main #1488

merged 15 commits into from
Jul 11, 2023

Conversation

blakewilson
Copy link
Contributor

Tasks

  • I have signed a Contributor License Agreement (CLA) with WP Engine.
  • If a code change, I have written testing instructions that the whole team & outside contributors can understand.
  • I have written and included a comprehensive changeset to properly document the changes I've made.

Description

Related Issue(s):

Testing

Screenshots

Documentation Changes

Dependant PRs

blakewilson and others added 15 commits June 14, 2023 13:28
* Fix next.js bundle comments

* Combine coverage reports into one

* Fix output directory

* Properly combine reports

* Use the proper script

* Merge coverage

* Merge coverage

* Delete old coverage file

* Revert coverage action changes
* fix: GetSeedNode `$id` is required

* fix: URI is also required in GetSeedNode

* Update seed-query.mdx

* Add changeset

* Grammar

---------

Co-authored-by: Blake Wilson <[email protected]>
* Feat: Add CoreButton, CoreButtons, Update example Deps.

* Chore: Add changeset

* Style: Correct button styles

* Chore: Update package-lock.json

* Chore: Added missing textAlign attribute.

* Feat: handle link target attribute

* Test: Testing Commit

* Test: Add Unit Tests fo CoreButtons

* Chore: Add missing testing styles

* Revert "Test: Testing Commit"

This reverts commit e71c34a.

* Chore: Remove testing globalStyleSheet
* Add changeset message

* Use correct link

* Link to the appropriate changeset usage documentation
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Bumps [semver](https://github.com/npm/node-semver) from 7.5.1 to 7.5.3.
- [Release notes](https://github.com/npm/node-semver/releases)
- [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md)
- [Commits](npm/node-semver@v7.5.1...v7.5.3)

---
updated-dependencies:
- dependency-name: semver
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [fast-xml-parser](https://github.com/NaturalIntelligence/fast-xml-parser) from 4.2.4 to 4.2.5.
- [Release notes](https://github.com/NaturalIntelligence/fast-xml-parser/releases)
- [Changelog](https://github.com/NaturalIntelligence/fast-xml-parser/blob/master/CHANGELOG.md)
- [Commits](NaturalIntelligence/fast-xml-parser@v4.2.4...v4.2.5)

---
updated-dependencies:
- dependency-name: fast-xml-parser
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Feat: Add CoreHeading block

* Chore: Add Changeset

* Test: Testing Commit

* Build: Update package-lock

* Revert "Test: Testing Commit"

This reverts commit 9dc1ba0.

* Security: Npm audit fix

* Update changeset-cli

* Security: Force semver overrides
* Initial corelist component

* Add reversed and start attributes

* Add testing

* Add tests cases

* Remove manual testing changes

* Add changeset

* npm audit fix

* Update packages/blocks/tests/blocks/CoreList.test.tsx

Co-authored-by: Theofanis Despoudis <[email protected]>

---------

Co-authored-by: Theofanis Despoudis <[email protected]>
* CoreImage file added

* CoreImage added to index file in blocks folder

* Removed BlockWithAttributes

* Stubbed jest tests for CoreImage

* Create chilled-eggs-begin.md

* Tinkering with styling test for CoreImage

* Fixed Linting Issue

* Next Image added- still tinkering

* Refactor CoreImage func to conditionally render

* Add Next as dependency to blocks package

* Troubleshooting caption testing coverage

* Update packages/blocks/src/blocks/CoreImage.tsx

Co-authored-by: Theofanis Despoudis <[email protected]>

* Update packages/blocks/src/blocks/CoreImage.tsx

Co-authored-by: Theofanis Despoudis <[email protected]>

* Update packages/blocks/src/blocks/CoreImage.tsx

Co-authored-by: Theofanis Despoudis <[email protected]>

* Update packages/blocks/src/blocks/CoreImage.tsx

Co-authored-by: Theofanis Despoudis <[email protected]>

* Fixed changeset reference

* Added anchor attribute as figure id

* Changed id to ImageId to avoid collisions

* Added ternary operation for linkTarget variable

* Unit testing tinkering

* Removed id per feedback, added anchor to Image

* Fixed test for caption in CoreImage
Co-authored-by: Blake Wilson <[email protected]>

* Create href helper

* Setup Core Image attributes

* Add tests for Next.js image and regular image

* Fix anchor, add testing instructions

* Add link to img test

* Remove testing steps

* Remove NextLink.tsx

---------

Co-authored-by: Theofanis Despoudis <[email protected]>
Co-authored-by: Blake Wilson <[email protected]>
…scaped. (#1478)

* Fix(@faustwp/core):  useLogin access token `code` is not properly escaped

* Chore: Changeset

* Update angry-papayas-applaud.md
Bumps [tough-cookie](https://github.com/salesforce/tough-cookie) from 4.1.2 to 4.1.3.
- [Release notes](https://github.com/salesforce/tough-cookie/releases)
- [Changelog](https://github.com/salesforce/tough-cookie/blob/master/CHANGELOG.md)
- [Commits](salesforce/tough-cookie@v4.1.2...v4.1.3)

---
updated-dependencies:
- dependency-name: tough-cookie
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@blakewilson blakewilson requested a review from a team as a code owner July 11, 2023 18:52
@changeset-bot
Copy link

changeset-bot bot commented Jul 11, 2023

⚠️ No Changeset found

Latest commit: bc7ab0d

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

@headless-platform-by-wp-engine

Check out the recent updates to your Atlas preview environment:

App Environment URL Build
faustjs preview-env-canary https://ha…wered.com ✅ (logs)

Learn more about preview environments in our documentation.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 11, 2023

Coverage report for packages/faustwp-core

St.
Category Percentage Covered / Total
🟢 Statements 83.56% 849/1016
🟡 Branches 68.53% 516/753
🟢 Functions 84.4% 119/141
🟢 Lines 83.21% 818/983

Test suite run success

122 tests passing in 22 suites.

Report generated by 🧪jest coverage report action from bc7ab0d

@github-actions
Copy link
Contributor

github-actions bot commented Jul 11, 2023

Coverage report for packages/faustwp-cli

St.
Category Percentage Covered / Total
🟢 Statements 84.71% 72/85
🔴 Branches 57.14% 8/14
🟡 Functions 63.64% 14/22
🟢 Lines 83.58% 56/67

Test suite run success

15 tests passing in 5 suites.

Report generated by 🧪jest coverage report action from bc7ab0d

@blakewilson blakewilson merged commit bbf1d5c into main Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants