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

Introduce a new plugin template structure. #37

Merged
merged 1 commit into from Jun 25, 2017

Conversation

guzzilar
Copy link
Contributor

@guzzilar guzzilar commented Jun 25, 2017

1. Objective

This concept introduces a new 'templates' structure that would make user/developer easily to work with in order to hook or extend the Omise template, by having a proper 'templates' folder (relocated from 'includes/templates') and separate an admin pages out from the user template.

2. Description of change

File path changes

Previous Current
includes/templates/omise-wp-admin-page.php includes/admin/views/page-dashboard.php
includes/templates/omise-wp-admin-transfer-page.php includes/admin/views/page-transfer.php
includes/templates/partials/header-template.php includes/admin/views/partials/header-template.php
includes/templates/partials/message-box-template.php includes/admin/views/partials/message-box-template.php
includes/templates/partials/transfer-box-template.php includes/admin/views/partials/transfer-box-template.php
includes/templates/omise-myaccount-panel.php templates/myaccount/my-card.php
includes/templates/omise-cc-form.php templates/payment/form-creditcard.php
includes/templates/omise-payment-form.php templates/payment/form.php

3. Quality assurance

πŸ”§ Environments:

  • Platform version: WooCommerce v3.0.8
  • PHP version: 7.0.16.

✏️ Details:

Tested if you can still access and see these pages.

3.1. βœ… Credit card form at the checkout page.
3.2. βœ… Saved card list panel at the 'my-profile' page.
3.3. βœ… Admin page, Omise dashboard.
3.4. βœ… Admin page, Omise transfer list page.

4. Impact of the change

  • Some part of localization will be broken.
  • Any merchants who edited the Omise core template files will be effected.

5. Priority of change

Normal

6. Additional Notes

No.

This concept introduces a new 'templates' structure that allow user/developer to be able hook or extend into their store design, by having a proper 'templates' folder (relocated from 'includes/templates') and separate an admin page out from normal template.

Note: This commit just do relocate files and creating a new structure, those concepts haven't been implemented yet.
@guzzilar
Copy link
Contributor Author

πŸ‘

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant