Skip to content

refactor(i18n): use common messages for generic button labels - #7020

Merged
mergify[bot] merged 1 commit into
modrinth:mainfrom
chank-op:i18n/use-generic-strings
Aug 7, 2026
Merged

refactor(i18n): use common messages for generic button labels#7020
mergify[bot] merged 1 commit into
modrinth:mainfrom
chank-op:i18n/use-generic-strings

Conversation

@chank-op

@chank-op chank-op commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

commonMessages already has "Cancel", "Close", "Clear" and "Success",
but a few components were defining their own. Swapped them over and
removed the old ones.

Nothing changes — the defaultMessages were identical.

The locale diff is just pnpm prepr pruning the orphaned strings out of
every language.

A few components defined their own "Cancel", "Close", "Clear" and
"Success" descriptors when commonMessages already has them. Swapped
them over and removed the old ones.

Locale files come from intl:prune-local dropping the now-orphaned
strings across every language.
@modrinth-bot

Copy link
Copy Markdown
Member

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@Prospector

Copy link
Copy Markdown
Member

@Mergifyio queue

@mergify

mergify Bot commented Aug 6, 2026

Copy link
Copy Markdown

Merge Queue Status

This pull request spent 7 hours 45 minutes 12 seconds in the queue, including 6 minutes 43 seconds running CI.

Required conditions to merge

@Prospector

Copy link
Copy Markdown
Member

@Mergifyio refresh

@mergify

mergify Bot commented Aug 7, 2026

Copy link
Copy Markdown

refresh

✅ Pull request refreshed

@mergify
mergify Bot merged commit 8aa22b9 into modrinth:main Aug 7, 2026
12 of 14 checks passed
@mergify mergify Bot removed the queued label Aug 7, 2026
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