There's no reason to have dollar signs in the salsa entity text.

The currency is country specific.

CommentFileSizeAuthor
#1 salsa_entity_currency_2160829.patch1.39 KBmiro_dietiker

Comments

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Active » Needs review
StatusFileSize
new salsa_entity_currency_2160829.patch1.39 KB

Attached patch should fix it.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

This is modelled after the salsa forms, which have the same label. salsa_commerce AFAIK replaces it with the default currency or remove it. Or so I thought.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented

It seems not to be replaced correctly then.

Move to salsa_commerce queue then?

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

1: salsa_entity_currency_2160829.patch queued for re-testing.

s_leu’s picture

s_leu commented
Status: Needs review » Reviewed & tested by the community

Looks and reads like RTBC to me

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

Let's commit it for now.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.