Skip to content

Fix currency price parsing and formatting - #2410

Open
przemo20 wants to merge 2 commits into
IsThereAnyDeal:developfrom
przemo20:patch-18
Open

przemo20 wants to merge 2 commits into
IsThereAnyDeal:developfrom
przemo20:patch-18

Conversation

@przemo20

@przemo20 przemo20 commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

This updates the currency formatting settings and fixes price parsing when a price contains multiple thousand separators.

This should fix the incorrect total price displayed when clicking the Select Unowned DLC button on pages such as https://store.steampowered.com/app/1222670/The_Sims_4/?cc=vn. You need to open this page while logged out of Steam (or in a private window) for the cc parameter to work. The total should now be displayed as 32.077.000₫ instead of ₫32,077.00.

The currency map still isn't perfect, as Valve formats prices differently across some pages. For example, on the https://store.steampowered.com/app/2691920/MolCollabo_v2/?cc=fr store page, the price is displayed as 1 473,--€, whereas on the https://store.steampowered.com/search/?sort_by=Price_DESC&ndl=1&cc=fr search page, it's displayed as 1 473,00€. On the Community Market, the currency symbol can also appear to the left of the amount.

Update 1

I forgot to mention that in all countries using USD except the USA, USD is also appended to the price, e.g. $1,605.86 USD.
obraz
(the price on the left shows how Augmented Steam currently formats the total, while the price on the right is the total displayed natively on the Steam website for all DLCs)

@przemo20

przemo20 commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor Author

I just noticed that this PR doesn't fix all price-parsing issues. For example, prices in Peruvian soles (PEN) are still parsed incorrectly. The currency symbol (S/.) contains a period, which is also used as the decimal separator, and appears before the amount.

For example: https://store.steampowered.com/app/223750/DCS_World_Steam_Edition/?cc=PE

obraz

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.

1 participant