Maybe an entire tab on every order for the gift wrapping message is a bit of overkill.

We could put it on the order itself (like Billing and Shipping information).

That way a customer can print the invoice including the gift wrapping message in one go.

Comments

msmithcti’s picture

Yep, I think you're right. I'll look into doing this.

summit’s picture

Version: » 7.x-1.x-dev

Hi,

Would love to see this implemented. Is this somewhere in the planning please?
Thanks a lot in advance for considering!
Greetings, Martijn

msmithcti’s picture

I've started working on this and it seems it isn't quite as simple as I thought, the problem is that the message is a field on the giftwrap line item not on the order. The order administrator view is meant to display the fields on an order so if we we're to show the giftwrapping info on the order page we'd have to 'tack' it on.

I'm also not sure the order page is designed to be printed out. Do any of these modules help?

silkogelman’s picture

Printing will probably a separate issue, maybe we should not worry about it in this issue?

Commerce PDF Invoice is basically a View made with the views_pdf module, so field information is easily added to it.

msmithcti’s picture

If the printing can be solved with the Commerce PDF Invoice what other use case is there for doing this? I agree a whole tab is overkill at the moment but it sets us up nicely for some of the future features that have been discussed.

silkogelman’s picture

What giftwrap options would you put in a separate tab?
Or actually: what usecases would you cover doing that?

The most basic usecase 'wrap this order as a gift: yes/no/deluxe' doesn't require a tab, and
Information about giftwrap per product could find its place into the line items of the order.
Adding a column or sub-line for the giftwrapping option could look very neat.
And by the looks of it the most flexible solution to do that seems to be the commerce_custom_product module, so if you'd ask me I'd say all kinds of per product giftwrap cases are already covered.

summit’s picture

Hi,
I think this module should be targeted on gift-wrapping per order and not per product. Per product stuff becomes more complicated and then may be commece_custom_product module comes into place. Right?
Greetings, Martijn

silkogelman’s picture

From what I know that sounds very logical.

To be sure I've created an issue in the Commerce Custom Product Module issue queue to ask if people know of a module that can do Commerce Custom Product module stuff at the order level.
#1668396: Commerce Customizable Order?

msmithcti’s picture

Title: Gift Wrapping message in tab on order? » Display giftwrap information on the order instead of in its own tab
Category: feature » task

I've not had chance to have a good look at the custom product module but if you guys say it's able to handle per product giftwrap there's no need to duplicate it here. That definitely negates the need for a separate tab (although as @s1l pointed out, there probably wouldn't be a need for it anyway).

I'll start working on this as and when I can.

summit’s picture

Hi,

I think there is still a need for this module for giftwrapping on order-level, like the current installation.
It is working great for anonymous users and thats my first focus. So please let that functionality stay in this module!

greetings, Martijn

msmithcti’s picture

I was only referring to the per product functionality - the more I think about it I don't think this module will end up going down a per-product route. There would have to be so many changes a better solution would be a separate module or to use an existing module (as discussed above).

sHaDoW_LiGhT’s picture

Hi,

I agree with this issue, having the gift wrapping information on the order page is more practical both for the administrator and the customer as he can see the informations in order history.

Any release planned ?