Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unconfirmed_link.vue unconfirmed_links_open_button.vueを削除 #7725

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

naokinaokiboo
Copy link
Contributor

@naokinaokiboo naokinaokiboo commented Apr 23, 2024

Issue

概要

Issueの内容としては、unconfirmed_link.vue,unconfirmed_links_open_button.vueの非Vue化でしたが、現在使用されていなかったため、それぞれ以下の対応になりました。

unconfirmed_link.vue

どこからも参照されていなかったため削除しました。

unconfirmed_links_open_button.vue

app/javascript/products.vueから参照されていましたが、unconfirmed_links_open_button.vueの使用箇所は、既にReactに置き換えられていました。(app/javascript/products.vueはメンターのダッシュボードには使用されているが、提出物ページとしては使用されておらず不要なコードが残った状態)

そのため、unconfirmed_links_open_button.vueを削除し、app/javascript/products.vueの不要コードを削除・整理しました。

変更確認方法

  1. chore/remove-unconfirmed-link-vueをローカルに取り込む
  2. ファイルの削除、整理内容が適切か確認する
    • まだ不要な部分が残っていないか、必要な部分を消してしまっていないか等を確認して頂きたいです

Screenshot

  • 不要ファイル、コードの削除のみなので無し

@naokinaokiboo
Copy link
Contributor Author

@goruchanchan
お疲れ様です。
ご都合の良いタイミングでレビューをお願いできますでしょうか🙏(急ぎではありません)
もしご都合悪いなどありましたら、その旨、仰っていただけたらと思います。
よろしくお願い致します。🙇‍♂️

@goruchanchan
Copy link
Contributor

@naokinaokiboo レビュー依頼ありがとうございます!一週間を目安に確認いたしますので、今しばらくお待ちください🙇‍♂️

Copy link
Contributor

@goruchanchan goruchanchan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@naokinaokiboo

お待たせしました🙏確認しました🙇‍♂️

まだ不要な部分が残っていないか、必要な部分を消してしまっていないか等を確認して頂きたいです

必要部分については、一通り関係しそうなページが表示されることを確認し、問題ないことを確認しました🙇‍♂️

LGTM と思いますので承認いたします🙇‍♂️

@naokinaokiboo
Copy link
Contributor Author

@goruchanchan
レビューありがとうございました!😄

@komagata
チームメンバーによるレビューが完了しましたので、レビューよろしくお願い致します🙏

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.

None yet

2 participants