Active
Project:
Commerce Giftwrap
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
7 May 2012 at 11:47 UTC
Updated:
10 Oct 2013 at 12:35 UTC
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
Comment #1
msmithcti commentedYep, I think you're right. I'll look into doing this.
Comment #2
summit commentedHi,
Would love to see this implemented. Is this somewhere in the planning please?
Thanks a lot in advance for considering!
Greetings, Martijn
Comment #3
msmithcti commentedI'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?
Comment #4
silkogelman commentedPrinting 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.
Comment #5
msmithcti commentedIf 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.
Comment #6
silkogelman commentedWhat 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.
Comment #7
summit commentedHi,
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
Comment #8
silkogelman commentedFrom 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?
Comment #9
msmithcti commentedI'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.
Comment #10
summit commentedHi,
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
Comment #11
msmithcti commentedI 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).
Comment #12
sHaDoW_LiGhT commentedHi,
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 ?