IceCubesApp icon indicating copy to clipboard operation
IceCubesApp copied to clipboard

Minor bug: "1 new posts" incorrectly pluralised

Open henrik opened this issue 2 years ago • 0 comments
trafficstars

Hi! Love the app.

The new-post notification says "1 new posts" instead of "1 new post".

It seems to be here in the code: https://github.com/Dimillian/IceCubesApp/blob/997a33df697e940a5310b01e1304eace1a7be60f/Packages/Timeline/Sources/Timeline/TimelineViewModel.swift#L52

I found this about pluralisation in Xcode: https://developer.apple.com/documentation/xcode/localizing-strings-that-contain-plurals

There seem to be a few more always-pluralised strings in https://github.com/Dimillian/IceCubesApp/blob/997a33df697e940a5310b01e1304eace1a7be60f/IceCubesApp/Resources/Localization/en.lproj/Localizable.strings.

henrik avatar Jan 20 '23 18:01 henrik