Skip to content

Call a shop 店舗 in Japanese - #151

Merged
dadachi merged 1 commit into
mainfrom
fix/japanese-shop-term
Oct 11, 2026
Merged

dadachi merged 1 commit into
mainfrom
fix/japanese-shop-term

Conversation

@dadachi

@dadachi dadachi commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • config/locales/ja.yml mixed 店舗名 (the shop's name attribute) with ショップ in four other places, so one error could say 店舗名 and the next ショップ. All four now say 店舗, as myturntagapi does (#210 there):
    • activerecord.models.shop
    • item_tag.shop
    • errors.messages.limit_count_shop
    • default_shop_description
  • test/i18n/japanese_messages_test.rb fails on any ショップ in the Japanese translations.

nativeapptemplate-agent

  • No change in behavior. scripts/ruby/rename.rb rewrites only the English spellings of the renamed models (Shop/shop/shops…) in .yml files. The YAML keys still rename together with the model, and the Japanese values stay as they are, just as ショップ did before.
  • As before, a generated app's Japanese messages still name the original noun (now 店舗).

Testing

  • I saw the new test fail before changing ja.yml.
  • bin/rails test: 584 runs, 0 failures. rubocop and brakeman are clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WyUzNEPWyhjjWrBcSaXSW4

ja.yml mixed 店舗名 (the shop's name attribute) with ショップ elsewhere, so a
validation error could say 店舗名 and a limit error ショップ. Use 店舗
throughout, as myturntagapi does, and fail a test on any ショップ.

The agent's rename.rb rewrites only the English spellings of Shop, so
these Japanese values are unaffected, as before.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WyUzNEPWyhjjWrBcSaXSW4
@dadachi
dadachi merged commit 1b04d14 into main Oct 11, 2026
3 checks passed
@dadachi
dadachi deleted the fix/japanese-shop-term branch October 11, 2026 05:54
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