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

Add/features module #1067

Merged
merged 29 commits into from
May 29, 2024
Merged

Add/features module #1067

merged 29 commits into from
May 29, 2024

Conversation

circlecube
Copy link
Member

Proposed changes

This adds the features module and updates the staging and performance module to integrate with the features module. Their initialization now uses the features setup according to the platform context.

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

add features to runtime and adjust menus accordingly
* develop: (25 commits)
  Composer(deps): Bump newfold-labs/wp-module-notifications
  Reorg deps
  Add notification with 'bluehost-app-nav' context in the app nav
  Composer(deps): Bump newfold-labs/wp-module-activation
  Composer(deps): Bump newfold-labs/wp-module-performance
  updating lock files
  Rename test file name
  Remove facebook module from composer packages
  fix border styles
  Change launchpad page slug
  ecommerce version bump to 1.3.28
  bug fixes
  fixing cypress test error
  cypress tests added
  coments addresed
  runtime version update and eslint fix
  update to products and desc
  lock files
  storepage
  redirection urls
  ...

# Conflicts:
#	composer.json
#	composer.lock
#	inc/Admin.php
@circlecube circlecube self-assigned this May 3, 2024
@circlecube circlecube marked this pull request as draft May 3, 2024 16:44
* main:
  Updated tested up to wp version
  Bump deactivation module 1.1.2
  update wordpress versions in cypress matrix
  Skip ecomm tests when the env is less than wp 6.4
  Bump npm package version
  Resolve conflicts
  Fix tests
  Fix tests
  update lock file
  Update Onboarding to 2.3.0
  update ecommerce version bump and lock files updated
  Bump ecommerce npm package
  Bump version
  bump ecommerce, onboarding and patterns modules

# Conflicts:
#	composer.json
#	composer.lock
* main:
  bump version in package file too
  bump plugin version to 3.10.2
  composer bump
  Update dependencies
  Update Onboarding to 2.3.1
Copy link

cypress bot commented May 21, 2024

2 failed and 1 flaky tests on run #8426 ↗︎

2 389 49 2 Flakiness 1

Details:

remove unused icon for route
Project: Bluehost Brand Plugin Commit: f9021a903d
Status: Failed Duration: 27:11 💡
Started: May 29, 2024 5:37 PM Ended: May 29, 2024 6:04 PM
Failed  Site-Capabilities/siteCapabilities.cy.js • 1 failed test

View Output Video

Test Artifacts
Verify Wondercart accessibility as per site capabilities > Verify clicking Install Now successfully installs Wonder Cart plugin Test Replay Screenshots Video
Failed  Store/storePage.cy.js • 1 failed test

View Output Video

Test Artifacts
Store Page- WooCommerce is deactivated/uninstalled > Verify that Payments tab is not displayed Test Replay Screenshots Video
Flakiness  vendor/newfold-labs/wp-module-ecommerce/tests/cypress/integration/Home/commerceHomePage.cy.js • 1 flaky test

View Output Video

Test Artifacts
Commerce Home Page- Next Steps > Verify Next steps when experience level is novice Test Replay Screenshots Video

Review all test suite changes for PR #1067 ↗︎

also adds a `featureToggle` helper method to interface with the feature API
* develop: (23 commits)
  fixing linting
  resolved conflicts
  ecommerce version bump 1.3.31
  update lock files
  lock file update
  update lock files
  added translation
  added translations
  updated lock files
  updating the class name to social
  update lock files
  eslint fix
  package.json update
  updated facebook changes
  lock files
  remove auto reload
  fix:imports
  align items to center
  fix:fb color
  plugin view fb connect
  ...

# Conflicts:
#	composer.json
#	composer.lock
#	src/app/pages/settings/index.js
1.9.0 is used in place of a 2.0.0 release for patterns and performance modules since the 1.x version is a cross-dependency for onboarding/onboarding-data modules.
* develop:
  Update lock file
  Update Onboarding to 2.3.2

# Conflicts:
#	composer.json
#	composer.lock
both as 1.9.9 since onboarding still requires 1.x versions of each module
@circlecube circlecube marked this pull request as ready for review May 29, 2024 16:56
Copy link

cypress bot commented May 29, 2024

2 failed and 1 flaky tests on run #8425 ↗︎

2 389 49 2 Flakiness 1

Details:

Merge a2394f1 into 63db968...
Project: Bluehost Brand Plugin Commit: 27be4d1d15 ℹ️
Status: Failed Duration: 25:46 💡
Started: May 29, 2024 4:58 PM Ended: May 29, 2024 5:24 PM
Failed  Site-Capabilities/siteCapabilities.cy.js • 1 failed test

View Output Video

Test Artifacts
Verify Wondercart accessibility as per site capabilities > Verify clicking Install Now successfully installs Wonder Cart plugin Test Replay Screenshots Video
Failed  Store/storePage.cy.js • 1 failed test

View Output Video

Test Artifacts
Store Page- WooCommerce is deactivated/uninstalled > Verify that Payments tab is not displayed Test Replay Screenshots Video
Flakiness  vendor/newfold-labs/wp-module-ecommerce/tests/cypress/integration/Home/commerceHomePage.cy.js • 1 flaky test

View Output Video

Test Artifacts
Commerce Home Page- Next Steps > Verify Next steps when experience level is novice Test Replay Screenshots Video

Review all test suite changes for PR #1067 ↗︎

@circlecube circlecube merged commit 79a5e93 into develop May 29, 2024
2 of 4 checks passed
@circlecube circlecube deleted the add/features-module branch May 29, 2024 21:28
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