search

tsharman / paprika

3 stars 2 forks source link

Order code should be automatically generated #22

Closed tsharman closed 12 years ago

tsharman commented 12 years ago

For Orders we should automatically generate the order code. You can just hash the name and email or phone number or whatever.

Remove the field in the "create order" dialog.

ekoschik commented 12 years ago

The only reason we want an order code is to tie in our order with an order on other software (like all data manage). I think that the user should be able to turn on or off whether or not there is a prompt for an order number on the new order screen.

tsharman commented 12 years ago

I just dont think the user should ever be creating an order code. It should always be automagically generated.

prashanthsadasivan commented 12 years ago

Personally, I just don't get what the need for an order code is. I think we should talk with Packard Auto about whether its needed at all, or if they do, Evan and I agree that it should be toggle able and entered by the business. Why the hell would a business want to memorize/enter a SHA1 hash? Keyword search is definitely more valuable than order code

tsharman commented 12 years ago

For now its not necessary but some businesses may want a unique key (maybe like 6 chars long) that their customers could use to look up their order.

Its similar to how airlines have eticket and confirmation numbers.

tsharman commented 12 years ago

Also, get yourselves some Github profile pics.

prashanthsadasivan commented 12 years ago

I disagree, it shouldn't be made randomly by us. Then businesses will have to know both the all data manage order number, and the paprika order number.

tsharman commented 12 years ago

Alright, lets just get rid of it then. @ekoschik can you remove it from the model and push the migrations?

Just add "fixes #22" anywhere in your commit message and it will comment and close this issue with a commit.

ekoschik commented 12 years ago

Ok I can take it out. In the future, we could have a switch on the accounts page allowing business to add an extra field like that if it works better on their side of things... just a thought. maybe if any of the businesses ask for it.

ekoschik commented 12 years ago

Order code has been removed