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

Release/3.8.4 #927

Merged
merged 42 commits into from
Feb 2, 2024
Merged

Release/3.8.4 #927

merged 42 commits into from
Feb 2, 2024

Conversation

circlecube
Copy link
Member

@circlecube circlecube commented Jan 31, 2024

Proposed changes

Module updates as follows:

  • Coming Soon: 1.1.18 => 1.2.0
  • Ecommerce: 1.3.18 => 1.3.21
  • Install Checker: 1.0.2 => 1.0.3
  • Onboarding: 1.12.0 => 1.12.1
  • Runtime: 1.0.7 => 1.0.9
  • Secure Passwords: 1.1 => 1.1.1
  • CTB: 1.1.2 => 1.1.3
  • Notifications: 1.2.2 => 1.2.3
  • Marketplace: 2.2.0 => 2.2.1
  • Global CTB: 1.0.9 => 1.0.10

Also includes various minor updates for packages - mainly Wordpress packages. Some linting fixes. Updates build to use node v20.

Type of Change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • Linting and tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

dependabot bot and others added 30 commits January 29, 2024 15:13
Bumps [newfold-labs/wp-module-onboarding](https://github.com/newfold-labs/wp-module-onboarding) from 1.12.0 to 1.12.1.
- [Release notes](https://github.com/newfold-labs/wp-module-onboarding/releases)
- [Commits](newfold-labs/wp-module-onboarding@1.12.0...1.12.1)

---
updated-dependencies:
- dependency-name: newfold-labs/wp-module-onboarding
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
…fold-labs/wp-module-onboarding-1.12.1

Composer(deps): Bump newfold-labs/wp-module-onboarding from 1.12.0 to 1.12.1
* develop:
  Composer(deps): Bump newfold-labs/wp-module-onboarding
update node version in plugin, nvm and main workflows
Bumps [actions/cache](https://github.com/actions/cache) from 3 to 4.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@v3...v4)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [react-use](https://github.com/streamich/react-use) from 17.4.3 to 17.5.0.
- [Release notes](https://github.com/streamich/react-use/releases)
- [Changelog](https://github.com/streamich/react-use/blob/master/CHANGELOG.md)
- [Commits](streamich/react-use@v17.4.3...v17.5.0)

---
updated-dependencies:
- dependency-name: react-use
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [@wordpress/dom-ready](https://github.com/WordPress/gutenberg/tree/HEAD/packages/dom-ready) from 3.49.0 to 3.50.0.
- [Release notes](https://github.com/WordPress/gutenberg/releases)
- [Changelog](https://github.com/WordPress/gutenberg/blob/trunk/packages/dom-ready/CHANGELOG.md)
- [Commits](https://github.com/WordPress/gutenberg/commits/@wordpress/[email protected]/packages/dom-ready)

---
updated-dependencies:
- dependency-name: "@wordpress/dom-ready"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [@wordpress/html-entities](https://github.com/WordPress/gutenberg/tree/HEAD/packages/html-entities) from 3.49.0 to 3.50.0.
- [Release notes](https://github.com/WordPress/gutenberg/releases)
- [Changelog](https://github.com/WordPress/gutenberg/blob/trunk/packages/html-entities/CHANGELOG.md)
- [Commits](https://github.com/WordPress/gutenberg/commits/@wordpress/[email protected]/packages/html-entities)

---
updated-dependencies:
- dependency-name: "@wordpress/html-entities"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
…op/actions/cache-4

GitHub Actions(deps): Bump actions/cache from 3 to 4
…/react-use-17.5.0

NPM(deps): Bump react-use from 17.4.3 to 17.5.0
…/wordpress/dom-ready-3.50.0

NPM(deps): Bump @wordpress/dom-ready from 3.49.0 to 3.50.0
…/wordpress/html-entities-3.50.0

NPM(deps): Bump @wordpress/html-entities from 3.49.0 to 3.50.0
  - Upgrading newfold-labs/wp-module-ecommerce (v1.3.18 => v1.3.20)
  - Upgrading newfold-labs/wp-module-runtime (v1.0.7 => v1.0.9)
Bumps [@wordpress/env](https://github.com/WordPress/gutenberg/tree/HEAD/packages/env) from 9.1.0 to 9.2.0.
- [Release notes](https://github.com/WordPress/gutenberg/releases)
- [Changelog](https://github.com/WordPress/gutenberg/blob/trunk/packages/env/CHANGELOG.md)
- [Commits](https://github.com/WordPress/gutenberg/commits/@wordpress/[email protected]/packages/env)

---
updated-dependencies:
- dependency-name: "@wordpress/env"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [@wordpress/icons](https://github.com/WordPress/gutenberg/tree/HEAD/packages/icons) from 9.40.0 to 9.41.0.
- [Release notes](https://github.com/WordPress/gutenberg/releases)
- [Changelog](https://github.com/WordPress/gutenberg/blob/trunk/packages/icons/CHANGELOG.md)
- [Commits](https://github.com/WordPress/gutenberg/commits/@wordpress/[email protected]/packages/icons)

---
updated-dependencies:
- dependency-name: "@wordpress/icons"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [@reduxjs/toolkit](https://github.com/reduxjs/redux-toolkit) from 2.0.1 to 2.1.0.
- [Release notes](https://github.com/reduxjs/redux-toolkit/releases)
- [Commits](reduxjs/redux-toolkit@v2.0.1...v2.1.0)

---
updated-dependencies:
- dependency-name: "@reduxjs/toolkit"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [cypress](https://github.com/cypress-io/cypress) from 13.6.3 to 13.6.4.
- [Release notes](https://github.com/cypress-io/cypress/releases)
- [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md)
- [Commits](cypress-io/cypress@v13.6.3...v13.6.4)

---
updated-dependencies:
- dependency-name: cypress
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Added links to Coming Soon page
…/wordpress/icons-9.41.0

NPM(deps): Bump @wordpress/icons from 9.40.0 to 9.41.0
…/wordpress/env-9.2.0

NPM Dev(deps-dev): Bump @wordpress/env from 9.1.0 to 9.2.0
…/cypress-13.6.4

NPM Dev(deps-dev): Bump cypress from 13.6.3 to 13.6.4
…/reduxjs/toolkit-2.1.0

NPM(deps): Bump @reduxjs/toolkit from 2.0.1 to 2.1.0
update ecommerce and runtime modules
- Upgrading newfold-labs/wp-module-coming-soon (1.1.18 => 1.2.0)
  - Upgrading newfold-labs/wp-module-install-checker (1.0.2 => 1.0.3)
  - Upgrading newfold-labs/wp-module-secure-passwords (1.1.0 => 1.1.1)
@circlecube circlecube requested a review from wpscholar January 31, 2024 18:56
@circlecube circlecube self-assigned this Jan 31, 2024
Copy link

cypress bot commented Jan 31, 2024

5 failed and 1 flaky tests on run #6898 ↗︎

5 328 24 0 Flakiness 1

Details:

update to latest ecommerce module in composer package file
Project: Bluehost Brand Plugin Commit: f3e0b6db1a
Status: Failed Duration: 22:49 💡
Started: Feb 2, 2024 2:56 PM Ended: Feb 2, 2024 3:19 PM
Failed  vendor/newfold-labs/wp-module-global-ctb/tests/cypress/integration/global-ctb.cy.js • 2 failed tests

View Output Video

Test Artifacts
Click to buy > CTB modal is functional Test Replay Screenshots Video
Click to buy > CTB fallback is functional Test Replay Screenshots Video
Failed  vendor/newfold-labs/wp-module-notifications/tests/cypress/integration/notifications.cy.js • 3 failed tests

View Output Video

Test Artifacts
Notifications > Test notification displays in plugin app with `all` Test Replay Screenshots Video
Notifications > Test notification displays in plugin app for specific page (settings) Test Replay Screenshots Video
Notifications > Dismissing notification removes it from the page Test Replay Screenshots Video
Flakiness  vendor/newfold-labs/wp-module-onboarding/tests/cypress/integration/3-ecommerce-onboarding-flow/get-started-experience.cy.js • 1 flaky test

View Output Video

Test Artifacts
Start Setup WP Experience Page > Check if events API call being made after radio buttons are clicked Test Replay Screenshots Video

Review all test suite changes for PR #927 ↗︎

Copy link

cypress bot commented Jan 31, 2024

9 failed and 1 flaky tests on run #6920 ↗︎

9 324 24 0 Flakiness 1

Details:

Merge f3e0b6d into 7bf44e0...
Project: Bluehost Brand Plugin Commit: 5a530675ab ℹ️
Status: Failed Duration: 24:47 💡
Started: Feb 2, 2024 2:59 PM Ended: Feb 2, 2024 3:23 PM
Failed  vendor/newfold-labs/wp-module-ecommerce/tests/cypress/integration/Store/storePage.cy.js • 1 failed test

View Output Video

Test Artifacts
Store Page- WooCommerce is deactivated/uninstalled > Verify Store page title and sub titles Test Replay Screenshots Video
Failed  vendor/newfold-labs/wp-module-global-ctb/tests/cypress/integration/global-ctb.cy.js • 2 failed tests

View Output Video

Test Artifacts
Click to buy > CTB modal is functional Test Replay Screenshots Video
Click to buy > CTB fallback is functional Test Replay Screenshots Video
Failed  vendor/newfold-labs/wp-module-notifications/tests/cypress/integration/notifications.cy.js • 3 failed tests

View Output Video

Test Artifacts
Notifications > Test notification displays in plugin app with `all` Test Replay Screenshots Video
Notifications > Test notification displays in plugin app for specific page (settings) Test Replay Screenshots Video
Notifications > Dismissing notification removes it from the page Test Replay Screenshots Video
Failed  vendor/newfold-labs/wp-module-onboarding/tests/cypress/integration/3-ecommerce-onboarding-flow/basic-info.cy.js • 3 failed tests

View Output Video

Test Artifacts
Basic Info Page > Check for the short URL tooltip & Modal exists when we use URL shortner Test Replay Screenshots Video
Basic Info Page > Check for twitter or instagram id starting with `@` to convert it to URL Test Replay Screenshots Video
Basic Info Page > Check if Social Media URL checks are done Test Replay Screenshots Video
Flakiness  vendor/newfold-labs/wp-module-ecommerce/tests/cypress/integration/Store/storePage.cy.js • 1 flaky test

View Output Video

Test Artifacts
Store Page- WooCommerce is deactivated/uninstalled > Verify Store and its sub tabs should have Install WooCommerce buttons Test Replay Screenshots Video

Review all test suite changes for PR #927 ↗︎

it seems to stall since it is the first test to run and we end up skipping out of onboarding on a fresh docker install
* main:
  Add `experimentalRunAllSpecs`
- Upgrading newfold-labs/wp-module-ctb (1.1.2 => 1.1.3)
- Upgrading newfold-labs/wp-module-notifications (1.2.2 => 1.2.3)
- Upgrading newfold-labs/wp-module-marketplace (2.2.0 => 2.2.1)
- Upgrading newfold-labs/wp-module-global-ctb (1.0.9 => 1.0.10)
- Upgrading newfold-labs/wp-module-ecommerce (v1.3.20 => v1.3.21)
@circlecube
Copy link
Member Author

We're seeing some failing tests still but local tests pass when run individually and manual testing checks out as well. We'll need to do some more test investigations but in the meantime we can release this version, since we've done ample manual testing. This release does fix a stripe connection issue

Need to look into the global-ctb module and the notifications module tests, they are consistently failing when run as part of the set but pass individually. There is most likely a cypress test conflict with other tests, but the module code seems to be fine.

@circlecube circlecube merged commit ddbfcef into main Feb 2, 2024
2 of 48 checks passed
@circlecube circlecube deleted the release/3.8.4 branch February 2, 2024 15:50
@circlecube circlecube mentioned this pull request Feb 2, 2024
8 tasks
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.

2 participants