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

ユーザー一覧を非vue化したい #7403

Closed
komagata opened this issue Feb 19, 2024 · 9 comments
Closed

ユーザー一覧を非vue化したい #7403

komagata opened this issue Feb 19, 2024 · 9 comments

Comments

@komagata
Copy link
Member

vueを撤廃して普通のページ(非SPA)にしたい。
kaminariでページングすることになると思う。

将来的にはこれを元にHotwire化する予定。

@yocchan-git
Copy link
Contributor

@komagata さん
@machida さん

実装について相談があります!🙇‍♂️
ユーザ一覧の非vue化について、JSを使用してもよろしかったでしょうか?

というのも、フォローの実装について現状は非同期で処理をしているのですが、
JSのコードを使わないとページの先頭に戻ってしまい...😅

Image from Gyazo

こういった場合JSを使って実装してもよろしかったでしょうか?
それとも、リダイレクトする感じにした方がいいでしょうか?🤔

お手隙でご回答いただけますと嬉しいです🙏

@komagata
Copy link
Member Author

@yocchan-git バニラJSは使用しても大丈夫です〜

@yocchan-git
Copy link
Contributor

ありがとうございます!
では、その方向性で行かせて頂きます💪

@yocchan-git
Copy link
Contributor

yocchan-git commented Apr 12, 2024

@komagata さん
@machida さん

本番環境で確認いただいてもよろしいでしょうか?
具体的には、相談部屋一覧にアクセスして「絞り込み」を確認していただきたいです。

  • Enterを押して検索結果が表示されること(半角文字なら3文字以上・全角なら2文字以上で検索されます)

こちらのコメントと同じ内容です!
よろしくお願いいたします🙏

@machida
Copy link
Member

machida commented Apr 12, 2024

@yocchan-git @komagata

本番環境で確認いただいてもよろしいでしょうか?
具体的には、相談部屋一覧にアクセスして「絞り込み」を確認していただきたいです。

本番で確認しました👍


相談部屋の絞り込みは確認できたのですが、ユーザー一覧のページでユーザーアイコンに role に合わせた border の枠が付いていたのがなくなってしまっていました。

例えばこのページの場合は、メンターなので赤い枠が付きます。
https://bootcamp.fjord.jp/users?target=mentor

@yocchan-git
Copy link
Contributor

@machida さん

指摘ありがとうございます!ちょっと確認します

@yocchan-git
Copy link
Contributor

@machida さん
cc @komagata さん

1点まだ不具合がありまして、デザインなのでよろしければ対応いただけますと嬉しいです
対応して欲しい画面はユーザー一覧画面(/users)になります。

具体的には、一番下までスクロールした時に、ページネーションと「気になるユーザーをフォローしてみよう」の案内の下にフォローカードが来てしまうことです。
Image from Gyazo

お手数ですが、一番上に表示されるようにしていただけますでしょうか?

またページネーションがない場合(例えばフォロー一覧など/users?target=followings)はフォローしないの選択が表示されないのも修正いただけますと嬉しいです。

Image from Gyazo

@machida
Copy link
Member

machida commented Apr 16, 2024

おお、下とのかぶりは未確認でした!対応しますー

@machida machida self-assigned this Apr 16, 2024
@yocchan-git
Copy link
Contributor

本番環境の動作確認OKでした!
Closeします👍

@machida さん
こちらのお早い対応ありがとうございました🙏

@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

3 participants