-
Notifications
You must be signed in to change notification settings - Fork 382
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
Remove store address editing #10321
Remove store address editing #10321
Conversation
Should the invoice printing related stuff be removed as well? Or at least the copies parameter and javascript for auto printing it. |
24ab58c
to
85f5ec3
Compare
printing and shipping-only versions probably not needed either, without shipping being handled on our end 🤔 |
@@ -19,13 +19,8 @@ | |||
</div> | |||
@endif | |||
|
|||
@for ($i = 0; $i < $copies; $i++) | |||
@if ($i > 0) | |||
<div class='print-page-break'></div> |
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.
the class style can be removed
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.
I clearly forgot about the comment I made a bit ago
No longer supported, handling of physical item shipping has gone via shopify partners for a while now.
Extra validation that forces an error if a non-shopify shippable item tries to checkout.
Placeholders hardcoded for old invoices.
closes #9490