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を非vue化したい #7590

Closed
komagata opened this issue Mar 24, 2024 · 3 comments
Closed
Assignees

Comments

@komagata
Copy link
Member

komagata commented Mar 24, 2024

vueを使わないようにするためにタイトルになっているファイルを使っている箇所を普通のHTMLでの実装に変える。

元々Reactに変更する予定でしたが、最終的にはHotwireにする予定なので、VueもReactも使わない実装にしたいです。(普通にviewに表示する方式)

@naokinaokiboo
Copy link
Contributor

@komagata @machida

本Issueの対応方針について確認させて下さい🙏

調査内容

調査しましたところ、本Issueで対象になっている各vueファイルは現在使用されていないようです。

unconfirmed_link.vue

現在、どこからも参照されておらず使用されていないと考えられます。

以下のPRで参照していたファイル(app/javascript/components/reports.vue)が既に削除されているようです。

企業ページの日報一覧が重複して表示されている問題を修正した

unconfirmed_links_open_button.vue

現在、app/javascript/products.vueから参照されていました。

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

提出物ページのReact化

方針

本Issueの目的としては、以下の2つのvueファイルを削除した上で、app/javascript/products.vueの使用されていない部分のコードを整理・削除するということでよいでしょうか?

  • unconfirmed_link.vue
  • unconfirmed_links_open_button.vue

以上、ご確認よろしくお願い致します。🙏

@komagata
Copy link
Member Author

@naokinaokiboo 使われていない場合は削除でOKです~!

@naokinaokiboo
Copy link
Contributor

こちら、本番環境で動作確認済みのためcloseします。

@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants