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 hook, change hook location and change private $error to public $error #1469

Merged
merged 13 commits into from May 12, 2021
Merged

Conversation

handoyo
Copy link
Contributor

@handoyo handoyo commented Apr 24, 2021

I change the private $error to public $error in the checkout/address as i found out that i cannot add additional error message from our extension.

As for the checkout/confirm.tpl, i moved the hook due to i put the hook in the wrong location.

…rror

I change the private $error to public $error in the checkout/address as i found out that i cannot add additional error message from our extension.

As for the checkout/confirm.tpl, i moved the hook due to i put the hook in the wrong location.
…ng template

I added _title and _text within the label for as there are no way to know which one was the title and the price.
@BasaraABC BasaraABC requested a review from abolabo May 4, 2021 04:51
@BasaraABC BasaraABC added this to the 1.3.1 milestone May 4, 2021
handoyo added 10 commits May 7, 2021 11:56
I found out that in sale/order_invoice the product id was not set so i change the hook in the order_invoice.tpl to use product name instead of product id.
I confirm that your suggestion on adding additional data behind the telephone works
I remove the additional "_title" and "_text" from the label and add label id
I remove the additional "_title" and "_text" from the label for and add label id
I remove the additional "_title" and "_text" from the label for and add label id
add additional hook
@abolabo abolabo merged commit 929cf84 into abantecart:1.3.1 May 12, 2021
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.

None yet

3 participants