Skip to content

Fix the "Destory" set and "Guardian Shiled" item names - #521

Open
openmubr wants to merge 1 commit into
Ignies:mainfrom
openmubr:pr/item-name-typos
Open

openmubr wants to merge 1 commit into
Ignies:mainfrom
openmubr:pr/item-name-typos

Conversation

@openmubr

@openmubr openmubr commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

Webzen's item table spells the Destroy set "Destory" and the shield 6/20 "Guardian Shiled", and the player reads both on screen.

  • ItemName and tools/Item.txt use the right spelling.
  • The English, Portuguese and Spanish packs get name fixes for the set (8..11/47) through the existing itemNames mechanism; English also fixes 6/20. The English and Portuguese layers now carry itemNames.
  • Tests check that each fixed name lands on a record of each shipped pack and that no "Destory"/"Shiled" is left. The item and text goldens are updated on purpose (only the en and pt text hashes change).

vitest on src/i18n, versions/shared/items and src/controllers/assets passes (26 files, 140 tests); tsc shows only the existing install/cli errors.

Webzen's item table spells the Destroy set "Destory" and the shield
6/20 "Guardian Shiled", and the player reads both on screen.

- ItemName and tools/Item.txt use the right spelling.
- The English, Portuguese and Spanish packs get name fixes for the set
  (8..11/47) through the existing itemNames mechanism; English also
  fixes 6/20. English and Portuguese layers now carry itemNames.
- Tests check every fixed name lands on a record of each shipped pack
  and leaves no "Destory"/"Shiled" behind. The item and text goldens
  are updated on purpose (only en and pt text hashes change).
@openmubr
openmubr requested a review from Ignies as a code owner November 8, 2026 13:35
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