51 文言の日本語化 #59
51 文言の日本語化 #59
Conversation
|
|
|
国際化対応的なことは、今回作ろうとしているものに対して過剰すぎると判断しました。 |
|
国際化っていう観点というよりは、「一箇所に集約させることでメンテしやすくする」というメリットを思って言ってみました。 |
確かにそちらのほうが良さげなのでそちらで対応します 🙏 |
|
「今それをやるべき！！！」って意図で言ったわけじゃなく、そういう対応方法もありますよねっていう確認のために言ってみただけなので、判断は omakase します！（わたしは今ならどちらでもいいと思う） |
|
👍 |
|
一旦 |
| @@ -16,7 +16,7 @@ def create | |||
| @order_item = @order.order_items.build(order_item_params) | |||
|
|
|||
| if @order_item.save | |||
| redirect_to order_order_items_path(@order) , notice: 'Order item was successfully created.' | |||
| redirect_to order_order_items_path(@order) , notice: '注文しました' | |||
There was a problem hiding this comment.
order_order_items_path(@order) のあとのスペースが気になる...
There was a problem hiding this comment.
ここの修正を抜かしてしまった・・・ 😇
PR単体で出して修正します 🙇
|
シングルクォートとダブルクォートが混在してたりするのもチョット気になるんだけど、今 rubocop 切ってるのもそういうのを気にせずがんがんすすめることを前提にしてるはずなので、とりあえずこのままでいいかなと思います。 |
|
変数展開の無い文字列はシングルクォートで囲むべきと考えているので、その点は直します。 |
|
お手すきの際にマージお願いします 🙇 |
概要
#51 に関する対応です。
実施事項
英語で書かれた表現を日本語化しました。
備考
特になし