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

Q&A Answer部分の非Vue化の不具合修正 #8275

Merged
merged 6 commits into from
Jan 18, 2025

Conversation

reckyy
Copy link
Contributor

@reckyy reckyy commented Jan 15, 2025

Issue

概要

下記PRの動作確認をステージング環境で行っていたところ、以下の不具合を発見したため修正。

不具合

  • 回答のプレビューが、初回のページ読み込み時(およびリロード時)には反映されていない
  • 回答に画像の添付ができない
  • 回答が投稿順になっていない(不規則)
  • 回答を投稿した後、回答フォームのtextareaのサイズがそのまま(10行の回答を投稿したら、回答フォームの行数が10行のまま)& 改行しても高さが広がらない

また、 @ayu-0505 さんからtest/system/comments_test.rbの193行目に引っかかるとお知らせいただいたので併せて修正。
(コメントを入力後プレビューに変更してからコメントを投稿した際、新規投稿フォームにてプレビューからコメントフォームにアクティブが戻っているか、というテスト内容)

変更確認方法

  1. bugfix/answer-preview-and-insert-buttonをローカルに取り込む
  2. foreman start -f Procfile.dev
  3. 適当な質問のページにアクセス。
  4. 上記事象が起こらないことを順番に確認。

Screenshot

動画、画像共に、変更前変更後の順番です。

  • 回答のプレビューが、初回のページ読み込み時(およびリロード時)には反映されていない
2025-01-17.23.35.12.mov
2025-01-16.17.04.20.mov
  • 回答に画像の添付ができない
2025-01-17.23.34.15.mov
2025-01-17.23.22.02.mov
  • 回答が投稿順になっていない(不規則)
スクリーンショット 2025-01-17 23 33 28 スクリーンショット 2025-01-17 23 31 14
  • 回答を投稿した後、回答フォームのtextareaのサイズがそのまま & 改行しても高さが広がらない

この項目だけ、以下の順番です。
1つ目 : 回答を投稿した後、回答フォームのtextareaのサイズがそのまま
2つ目 : 改行しても高さが広がらない
3つ目 : 変更後(同時に上記2つの不具合を解消しています。)

2025-01-17.23.46.10.mov
2025-01-17.23.39.01.mov
2025-01-17.23.23.51.mov
  • コメントを入力後プレビューに変更してからコメントを投稿した際、新規投稿フォームにてプレビューからコメントフォームにアクティブが戻っているか
2025-01-17.23.42.14.mov
2025-01-17.23.22.59.mov

@reckyy reckyy self-assigned this Jan 15, 2025
@reckyy reckyy force-pushed the bugfix/answer-preview-and-insert-button branch from 8022214 to 689c69f Compare January 16, 2025 06:53
@reckyy reckyy force-pushed the bugfix/answer-preview-and-insert-button branch from 689c69f to dc549d3 Compare January 17, 2025 13:14
@reckyy reckyy changed the title Bugfix/answer preview and insert button Q&AのAnswer部分の非Vue化PRの不具合修正 Jan 17, 2025
@reckyy reckyy changed the title Q&AのAnswer部分の非Vue化PRの不具合修正 Q&A Answer部分の非Vue化の不具合修正 Jan 17, 2025
@reckyy reckyy marked this pull request as ready for review January 17, 2025 14:58
@reckyy
Copy link
Contributor Author

reckyy commented Jan 17, 2025

@komagata
お疲れ様です。
修正完了したので、お手隙の際にレビューをお願いしたいです。
よろしくお願いいたします 🙇

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit 0c1066d into main Jan 18, 2025
12 checks passed
@komagata komagata deleted the bugfix/answer-preview-and-insert-button branch January 18, 2025 05:04
@github-actions github-actions bot mentioned this pull request Jan 18, 2025
32 tasks
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.

2 participants