-
Notifications
You must be signed in to change notification settings - Fork 7
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
docs:Update plugin_quickstart.rst #68
Conversation
Thanks for the pull request, @AyeshaSiddique01! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. Once you've signed the CLA, please allow 1 business day for it to be processed. After this time, you can re-run the CLA check by adding a comment here that you have signed it. If the problem persists, you can tag the |
Hi @AyeshaSiddique01! Thanks for this contribution. Would you mind adding a description of your changes to the top of the pull request? Also, it looks like you're contributing on behalf of edly / arbisoft. Please have your manager reach out to [email protected] to have you added to their entity CLA agreement so we can get this pull request reviewed for you. Thank you! |
@AyeshaSiddique01 I notice there are some commit-lint failures. Please note that we use conventional commits across Open edX projects. You can read about the details here. Can you please amend your commit messages to follow our standard? |
Hi @AyeshaSiddique01! Just following up on this. |
Hi @AyeshaSiddique01! Checking to see if you plan to pursue this pull request? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for this contribution @AyeshaSiddique01.
I think the step by step guide to create products and creating a sale is very nice. However I would suggest to move it to the how-to section of the docs as this quickstart is mostly focused on getting the configuration right.
- PHP 8.0 or greater is recommended | ||
- Wordpress 6.3.1 or greater | ||
- `WooCommerce plugin. <https://wordpress.org/plugins/woocommerce>`_ | ||
- Setup https://github.com/edly-io/tutor-contrib-wordpress.git plugin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not a public repository (or it has been deleted since).
Also hosting wordpress via tutor is not a requirement. For the openedx-wordpress-ecommerce
the only requirements are as listed before.
I have already
Yes, I have a plan to pursue this pull. |
Sure |
@AyeshaSiddique01 I notice there are some commit-lint failures. Please note that we use conventional commits across Open edX projects. You can read about the details here. Can you please amend your commit messages to follow our standard? |
Hi @AyeshaSiddique01! I'm going to close this pull request for now, however, we can reopen if you'd like to pursue in the future. Thanks! |
@AyeshaSiddique01 Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future. |
The steps to enroll users in the course are updated.
We cannot enroll users in the Enrollment Management tab as there is no permission for anyone to enroll users in the course.