fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
281 stars 74 forks source link

bookmark削除のトーストの文言を日本語に変更 #7891

Open su-su-su-su opened 1 week ago

su-su-su-su commented 1 week ago

Issue

変更確認方法

  1. bug/toast-japanese-bookmark-deletionをローカルに取り込む
  2. foreman start -f Procfile.devでサーバーを立ち上げる
  3. kimuraでログイン(ブックマークがはじめからある為)
  4. ダッシュボードの最新のブックマークからブックマークを削除
  5. トーストの文言がカタカナでブックマークになっているか確認
  6. 最新のブックマークから登録している記事をクリック
  7. Bookmark中をクリック
  8. トーストの文言がカタカナでブックマークになっているか確認

Screenshot

変更前

ダッシュボードからブックマークを削除 スクリーンショット (577)

ブックマークした記事からブックマークを削除

4e3a5ce200c883837945e78e84632dc1

変更後

ダッシュボードからブックマークを削除 スクリーンショット (578)

ブックマークした記事からブックマークを削除

cf33d49cbbc83ccde082eda3427272be
su-su-su-su commented 1 week ago

@umizaru

お疲れ様です。 チーム開発に加わりましたsu-su-su-suと申します。 可能でしたらお手隙の際に、こちらのPRのレビューをお願いできますでしょうか? ご都合が合わない場合はおっしゃってください。 どうぞよろしくお願いいたします。

umizaru commented 1 week ago

@su-su-su-su 承知しました!この週末めどでレビューさせていただきます👍

su-su-su-su commented 6 days ago

@umizaru

お忙しいところありがとうございます。 よろしくお願いいたします🙇

umizaru commented 5 days ago

@su-su-su-su お待たせしました、レビューしました! @su-su-su-su さんが修正した箇所は問題ないようです👍

2点だけありまして、

以上です!ご確認よろしくお願いします🙏

su-su-su-su commented 5 days ago

@umizaru

お忙しい中レビュー頂きありがとうございます!

Commitsの数が38と他の方のPRも含まれてしまっていますが、これは問題ないでしょうか? PRのタイトルに不要な「に」が含まれているので削除したほうが良いかなと思いました。

ご指摘ありがとうございます。 確認不足でした。こちら修正ができましたので、お手隙の先に再度レビューお願い致します🙇