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 coming soon hooks from ecommerce module #18

Merged
merged 7 commits into from
Jan 11, 2024

Conversation

circlecube
Copy link
Member

This migrates the coming soon hooks from the e-commerce module into the coming soon module so the coming soon module can own the coming soon functionality, logic and components.

See: newfold-labs/wp-module-ecommerce#199 where these are removed from ecom.

includes/ComingSoon.php Outdated Show resolved Hide resolved
@circlecube circlecube marked this pull request as draft January 4, 2024 19:21
* main:
  remove only-module-tests flags
  test only-module-tests in all cases
  test `only-module-tests` workflow flag
  add .nvmrc as v16
  add brand plugin test workflow
  fix autofixable lint issues in test file
  Update Modal.jsx
  Update package.json
  Updated the approach to use the post status from wp.data
  Add translation
  Remove console.log
  Add pre-publish modal when coming soon is active

# Conflicts:
#	includes/ComingSoon.php
@circlecube
Copy link
Member Author

This addresses issue #6

@circlecube circlecube marked this pull request as ready for review January 5, 2024 22:15
* main:
  remove unnecessary postcss config file
  Add missing package-lock file and postcss config
  Update API docs
  Code comments
  Update `comingSoon.isEnabled` to return boolean value or null on error
  JavaScript API and API routes
  Add service provider class to the container
  Update README.md
  Add new API docs

# Conflicts:
#	package-lock.json
@wpalani wpalani merged commit 98f13af into main Jan 11, 2024
6 checks passed
@wpscholar wpscholar deleted the add/hooks-from-ecom-module branch January 11, 2024 20:30
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.

3 participants