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.

Sign up for GitHub

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

Jump to bottom

answer.vue, answers.vueを非vue化する #5116

Closed
komagata opened this issue Jun 30, 2022 · 24 comments
Closed

answer.vue, answers.vueを非vue化する #5116

komagata opened this issue Jun 30, 2022 · 24 comments
Assignees
@reckyy
Labels
5 雑事 非Vue/React化

Comments

@komagata
Copy link
Member

komagata commented Jun 30, 2022 •
edited
Loading

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

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

応相談

最終的にはrailsを7系にアップデートしてhotwireにする予定。
その前段階として、SPAではなく、極力JS無しの紙芝居構成にしたい。（hotwireを使うとその紙芝居構成を画面遷移無しで切り替えられる）

現状はGitHub風のコメント欄になっているが、一時的にもっと簡単な形式にしてもいいかもと思っているが @machida さんと相談して進めていただきたいです。
（上記の部分もcommentsのReact化を大変にしている要因の一つかもと思うので）
@github-actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Aug 30, 2022
@komagata komagata removed the stale label Aug 30, 2022
@komagata komagata changed the title answer.vue, answers.vueをVueMounterに対応する answer.vue, answers.vueをreactに対応する Sep 17, 2022
@github-actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Nov 17, 2022
@komagata komagata removed the stale label Nov 18, 2022
@github-actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Jan 18, 2023
@komagata komagata removed the stale label Jan 23, 2023
@github-actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Mar 25, 2023
@komagata komagata removed the stale label Mar 28, 2023
@github-actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label May 30, 2023
@komagata komagata removed the stale label Jun 1, 2023
@github-actions
Copy link

github-actions bot commented Aug 1, 2023

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Aug 1, 2023
@komagata komagata removed the stale label Aug 1, 2023
@github-actions
Copy link

github-actions bot commented Oct 2, 2023

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Oct 2, 2023
@komagata komagata removed the stale label Oct 3, 2023
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 3, 2023

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Dec 3, 2023
@komagata komagata removed the stale label Dec 5, 2023
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 4, 2024

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@reckyy
Copy link
Contributor

reckyy commented Aug 19, 2024 •
edited
Loading

@machida
お疲れ様です。
こちらのIssueの実装方針についてご相談させていただきたいです。

現状はGitHub風のコメント欄になっているが、一時的にもっと簡単な形式にしてもいいかもと思っているが @machida さんと相談して進めていただきたいです。
（上記の部分もcommentsのReact化を大変にしている要因の一つかもと思うので）

ベストアンサーの表示を簡略化する、リアクションをGitHubの賛成票のみにする、といった提案を私からするのではなく、machidaさんから簡略化したデザインをいただき、それを基に実装を進めるという方向が良いでしょうか。

ご存知かもしれませんが、GitHubの賛成票については以下の内容です。

スクリーンショット 2024-08-19 11 52 44

@machida
Copy link
Member

machida commented Aug 19, 2024

@reckyy
機能の簡略化は無しでいきましょう。というのも、今より機能を減らして喜ぶのは開発者だけですが、機能やサービスはユーザーのためにあるからです。今とそっくりそのまま実装してください。難しいところが出てきたら、都度相談してください。

Anawerの一覧に関しては、現在は数件ごとに読み込みをさせる方式ですが、そこはpagerでもいいと考えています。

@reckyy
Copy link
Contributor

reckyy commented Aug 19, 2024

@machida
承知しました！
引き続きよろしくお願いいたします。 🙇

@reckyy
Copy link
Contributor

reckyy commented Aug 19, 2024

📝メモ

answers.vueを内部で使用している、question-answers.vue
同じくquestion-answers.vueの内部で使用されている commentPlaceholder.vue ai_answer.vue
これらも同様に非Vue化する。

@komagata komagata moved this to 作業中 in bootcamp Aug 21, 2024
@reckyy reckyy mentioned this issue Aug 25, 2024
Merged
6 tasks
@reckyy
Copy link
Contributor

reckyy commented Aug 25, 2024 •
edited
Loading

@machida

お疲れ様です！
一点確認させていただきたいことがあります。

Anawerの一覧に関しては、現在は数件ごとに読み込みをさせる方式ですが、そこはpagerでもいいと考えています

この数件ごとに読み込みをさせる方式というのは、提出物のコメントで見られる以下の部分のことでしょうか。
スクリーンショット 2024-08-25 12 39 27

もし上記の認識が正しければ、今回非Vue化するファイルにはこの方式が適用されていなかったため、回答を全てそのまま羅列する形式でも問題ないでしょうか。（現在の仕様と同様、ひたすら下にスクロールする形）
画像の青の部分です。
image
#7439 より引用

よろしくお願いいたします。

@machida
Copy link
Member

machida commented Aug 26, 2024

その認識で正しいです！
読み込み方式を使わずにanswerを羅列する場合、answerが多いとスマホのSafariでエラーが出るので、読み込み方式でなく、ページャーでもいいので、一度に表示する件数の上限を15くらいに制限を入れていただきたいです。

@machida
Copy link
Member

machida commented Aug 26, 2024

IMG_2862
こういうエラーが出ない様にしたいです。

@reckyy
Copy link
Contributor

reckyy commented Aug 26, 2024 •
edited
Loading

@machida
ご返信ありがとうございます！

新たにpagerの処理も追加する旨、承知しました 💪
以下、新規回答フォームに関してです。もし変更した方がよければ、教えていただけますと幸いです。

もし回答数が15件以上あり、回答のページが複数あった場合、新規回答フォームは最後のページにのみ表示されるようにしようと思っています。
スクリーンショット_2024-08-26_16_14_27

スクリーンショット_2024-08-26_16_14_32

@reckyy
Copy link
Contributor

reckyy commented Aug 27, 2024

@machida

もし回答数が15件以上あり、回答のページが複数あった場合、新規回答フォームは最後のページにのみ表示されるようにしようと思っています。

と書きましたが、やっぱり各ページに新規回答フォームは配置しようと思います。

@machida
Copy link
Member

machida commented Aug 27, 2024

@reckyy 了解です！

やっぱり各ページに新規回答フォームは配置しようと思います。

そのようにお願いしますー

@github-actions GitHub Actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Oct 27, 2024
@reckyy reckyy removed the stale label Oct 30, 2024
@github-actions GitHub Actions
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Dec 30, 2024
@komagata komagata removed the stale label Jan 5, 2025
This was referenced Jan 16, 2025
Merged
Merged
@reckyy reckyy moved this from レビュー中 to 本番確認中 in bootcamp Feb 4, 2025
@reckyy
Copy link
Contributor

reckyy commented Feb 4, 2025

本番環境で、以下の動作を確認しました。

  • 新規回答
    • 回答数の更新
  • 既存の回答
    • 内容の更新ができること
    • ベストアンサーにできる
    • ベストアンサーを取り消す
    • 削除する。その際、以下の事項も確認
      • それがベストアンサーなら、ベストアンサーを取り消す
      • 回答数の更新
      • 解決済みなら、未解決に質問のステータスを変更
    • 日付をクリックすると、AnswerのURLを取得
    • AnswerのURLを開くと、その回答までジャンプすること
  • 見つけた不具合たち（解消されていたら、checkをいれています）
    • 回答のプレビューが初回のページ読み込み時（およびリロード時）には反映されていない
    • 回答に画像の添付ができない
    • 回答が投稿順になっていない
    • 回答を投稿した後、回答フォームのtextareaのサイズがそのまま（10行の回答を投稿したら、回答フォームの行数が10行のまま）& 改行しても高さが広がらない
    • コメントを入力後プレビューに変更してからコメントを投稿した際、新規投稿フォームにてプレビューからコメントフォームにアクティブが戻っていない
    • コンソールエラー
    • 回答がない場合、回答のローディングスケルトンがずっと表示される
    • watchしていない質問に回答した後、watch状態にならない

動作確認完了したため、本Issueはcloseいたします。

@reckyy reckyy closed this as completed Feb 4, 2025
@github-project-automation github-project-automation bot moved this from 本番確認中 to 完成 in bootcamp Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@reckyy reckyy

Labels
5 雑事 非Vue/React化
Projects
bootcamp
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@komagata @machida @reckyy