Skip to content

fix: use nullish coalescing instead of OR on fraction digits#5788

Merged
Prospector merged 1 commit intomodrinth:mainfrom
shellawa:main
Apr 12, 2026
Merged

fix: use nullish coalescing instead of OR on fraction digits#5788
Prospector merged 1 commit intomodrinth:mainfrom
shellawa:main

Conversation

@shellawa
Copy link
Copy Markdown
Contributor

for zero-decimal currencies like JPY or VND, this is returning 2 instead of 0

@modrinth-bot
Copy link
Copy Markdown
Member

modrinth-bot commented Apr 12, 2026

Note

This changelog has been baked. Any further edits will not be reflected.

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

  • Fixed certain currencies showing pricing with the incorrect number of digits.

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@triphora triphora requested a review from a team April 12, 2026 15:37
@Prospector Prospector added this pull request to the merge queue Apr 12, 2026
Merged via the queue into modrinth:main with commit a2a97d1 Apr 12, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants