Sorry Chris. That is not in there yet. Maybe add a feature request and we can add it - there would not be much too it.

I could always add it myself and submit a pull request if that would be ok with you?

That would be fantastic. Are you thinking a new table in the database?

It strikes me as just a field against each order, (though I don't yet know how orders are stored), so I wouldn't think a whole new table would be needed but i'll investigate further into the inner workings!

I'll need the feature before I can finish the site i'm working on so i'll definitely be putting something together. That'll be tomorrow however, since it's now 11pm here and I need some sleep!

I totally understand. I think a new table may be the most flexible as you could then stuff notes for Invoices, Orders, Shipments and Payments .... similar to the AuditLog setup.

Maybe ping me tomorrow before you get going to look more closely at your use case.

You can add a table Note related to order. After you need to add edit note into backoffice

Hi Biagio,

This sounds like a solution that wouldn't be fully integrated into Merchello. I'd rather add the feature directly into Merchello for a future version so that it would be usable out of the box.

Chris

Rusty:

I'm starting work on this now. My use case is simply for a customer to give extra instructions for delivery/picking etc but I can see the potential for other uses (e.g: communication between sales and warehouse) so I am going down the additional table route as suggested. As you suggested, the way the audit log works does seem like exactly the right place to start so i'm currently duplicating all of the audit log stuff and adapting it. I suspect i'll have something to look at today. Initially these will probably just be read only in the back office for sales since i'm very new to angular but once the framework is there it shouldn't be to hard to add read/write everywhere else.

Chris

Yes, I mean into Merchello...you need to create a plugin or customize the solution and push the code on github.

Hey Chris,

The angular will not be too bad for your use case. I'm guessing we can do it as a little directive that we can put on the invoice (sort of like the sales history directive). If you get stuck on that, don't worry.

Hi Rusty,

I'm still working on this. You can see my progress on my fork here: https://github.com/chris64digital/Merchello/commits/1.9.1

Can you take a look and make sure i'm going in the right direction please if you have time.

I'm struggling to work out how to get a note applied to an invoice at the moment, i've added a function to the salepreparation to add a new note but can't seem to figure out what to do next! I'm sure i'll work it out in the morning tomorrow though.

Thanks

Chris

Hi Chris,

I'll take a look this afternoon. Getting the note onto the invoice should not be a big deal. I think we'll wind up creating another task for the invoice Builder or something. That'll require a few adjustments to the invoice - but I can handle those.

     <taskChain alias="SalesPreparationInvoiceCreate">
  <tasks>
    <task type="Merchello.Core.Chains.InvoiceCreation.AddBillingInfoToInvoiceTask, Merchello.Core" />
    <task type="Merchello.Core.Chains.InvoiceCreation.ConvertItemCacheItemsToInvoiceItemsTask, Merchello.Core" />
    <task type="Merchello.Web.Workflow.InvoiceCreation.AddCouponDiscountsToInvoiceTask, Merchello.Web" />
    <task type="Merchello.Core.Chains.InvoiceCreation.ApplyTaxesToInvoiceTax, Merchello.Core" />        
    <task type="Merchello.Core.Chains.InvoiceCreation.ValidateCommonCurrency, Merchello.Core" />
    <task type="[ADD NOTES TASK]" />
  </tasks>

Hi Rusty,

I'm struggling a little to get the notes saved against the basket. They are serialized into xml just fine but when I attempt to deserialize them I get null reference errors which I can't get to the bottom of. I believe that it is something to do with the fact that i'm inheriting from entity and the exception references the OnPropertyChanged method. I'm not sure exactly how the entity system works. Am I right to be inheriting from Entity? Should I perhaps have a separate object for use in serializing/deserializing?

Thanks

Chris

Serializing/Deserializing for the Back Office?

If you post a link to your repo I can take a look.

Hi Rusty,

It's for the front end, before the basket is made into an invoice, using salepreparation. I am attempting to store the notes in the same way as the address is stored since, in my use case, the note is related to the shipping address.

My repo is here: https://github.com/chris64digital/Merchello

Thanks

Chris

Hi Trevor,

No, I have not yet finished working on this. I'll be back to it, hopefully today, to get it finished!

Chris

Hi Trevor,

I have more or less finished this now. You can try it out on my fork of 1.9.1 which can be seen here: https://github.com/chris64digital/Merchello/tree/1.9.1

There are some new methods on sale preparation for saving and fetching a note against the invoice. These are saved in memory until the invoice is saved to the database and show on the saleoverview page currently.

Chris

is working on a reply...