Skip to content
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

clarificiation of minimum order quantity - currency independant #838

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mjavurek
Copy link
Contributor

@mjavurek mjavurek commented Feb 23, 2021

The German translation for minimum order quantity is changed:

"Wert" repräsentiert für mich eher einen Geldbetrag als "Menge", daher wird die deutsche Übersetzung aus meiner Sicht klarer.

Additionally, a currency symbol is displayed for the minimum order quantity in following locations:

  • supplier edit form: as placeholder in the input field
  • supplier show
  • order form

"Wert" repräsentiert für mich eher einen Geldbetrag als "Menge", daher wird die deutsche Übersetzung aus meiner sicht klarer.
currency symbol as placeholder in minimum order quantity input field
Display min_order_quantity as currency
@mjavurek mjavurek changed the title Update de.yml for minimum order quantity - currency independant clarificiation of minimum order quantity - currency independant Mar 21, 2021
@wvengen
Copy link
Member

wvengen commented Jun 4, 2021

Thank you for your PR! I have been confused by the meaning of this. The field is called quantity, but at some point someone added some logic related to monetary amount. I have used it in the past for a textual description. And indeed, the field is a string, as can be seen in https://github.com/foodcoops/foodsoft/blob/master/db/schema.rb#L473. This is not an ideal situation.

Maybe we need to find clarity on its use in the (user) mailing-list. And introduce a monetary field separate from a 'free-form' quantity field (if it is used at all). Perhaps there are other uses (e.g. weight or crates or whatever).

@twothreenine
Copy link
Contributor

twothreenine commented May 31, 2022

I'd suggest adding a text field order_note to supplier, which prefills the order_note field of each order opened for this supplier. This could be used for various information like minimum order quantity in crates etc., transport cost, link to supplier's website etc. (Additionally, we could also extend the custom fields feature for data like this, but that would take more work.)

Then we could migrate minimum order quanitity to a monetary field (rename it to minimum order value?), detect all non-numeric data saved in it, and move it into order_note (with the respective locale of minimum order quantity).

@paroga
Copy link
Member

paroga commented Jun 3, 2022

I'd suggest adding a text field order_note to supplier, which prefills the order_note field of each order opened for this supplier.

What's the benefit compared to copying the order? If we add a order_note to the Order I'd prefer to add it in addition to the existing note.

I like the idea of extending the custom fields to the supplier and your migration path for existing data.

@twothreenine
Copy link
Contributor

twothreenine commented Jun 3, 2022

Sorry, I meant a field supplier.order_note which prefills / acts as a default for the order.note. (In addition to the existing supplier.note)

What's the benefit compared to copying the order? If we add a order_note to the Order I'd prefer to add it in addition to the existing note.

You wouldn't have to click "copy" each time and could open orders normally, while you would still be able to adapt it in case the conditions differ for a specific order. (Also, copying an order does not copy the note atm)
What's the benefit of an additional order.order_note field?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants