Description
[天下無双]自分の前後のランキングを表示 #670
[天下無双]自分の前後のランキングを表示 #670
@kouho @wada
コードレビューをお願い致します。
レイアウトに関してはあくまで仮になります。
@iwamoto
ランキングデータ作成する時って
・user_event_pt
・user_info
にデータを入れて、event_ranking_event_battle.incを実行してら良い?
@wada
あ、すみません
そのあたり記載なかったですね。
はい、それで大丈夫です
@iwamoto
確認しました。動作は問題なさそうです。
自分の色は、他のランキングと同じ黄色の背景色が良いかなと思います。※スクショ参照background-image:-webkit-gradient(linear,left top,left bottom,from(#ffef99),to(#ffd700));
@wada
承知しました。そちらで修正対応を行います。
@iwamoto @wada
すみません、いまいちよく分からずどういった動作になれば正と判断できるのでしょうか?
@kouho
僕の試した方法ですが、
1. 前回の天下無双のイベントptデータとユーザデータをローカルのdbにinsert(自分のuser_noとバッティングしないように注意)
user_event_pt.sql
user_info.sql
2. 自分のuser_event_pt.event_ptを確認したい順位あたりのポイント数量に変更
3. ランキングバッチを実行
ランキングデータが入り、自分の順位が圏外でなければ自分の順位ボタンが表示されます。
docker-compose exec web bash
cd /home/web/jr-web/lib/exec
ENVIRONMENT=develop PF=yamada HTTP_HOST=127.0.0.1 IS_DOCKER=true php event_ranking_event_battle.inc4. 1ページ目の順位、ヘッダーの変わっている順位、51位以降の順位をそれぞれ確認
(目的の順位に応じてuser_even_pt.event_ptを調整)
UIや自分の順位、TOPへ戻るボタンの挙動などを確認
これで、自分の順位ボタンから自分が表示されているページへ飛べて、自分の順位の上下に他のユーザーがいれば要件は満たしていると思います。
@wada
共有ありがとうございます。試してみます。
@iwamoto
こちらでも確認しました。動作上の問題はないかと思います。
@iwamoto
使い勝手の問題ですが、以下は時間があれば対応お願いしたいです。
・ランキングTOPのリンクは、自分の順位関係なく、2ページ以降常に表示して欲しい
・自分の順位ページに遷移した後、ページャーのボタンで戻ると上位ページが見えなくなることがある(自分の順位ページに遷移するとページングの位置がずれるため)
@wada
・自分の背景色変更
・TOPリンクの修正
こちらは対応しました。
ページャーの件は承知しています。
ページャーで戻る先が20位より下の場合は戻れなくなります。
ただ、20位より下はランキングTOPより確認できます。
(TOPリンクを修正したので、大丈夫ではないかとも考えますが…)
@iwamoto
修正内容確認しました。OKです。
ページャーはこのままでも大丈夫です。
実装前に仕様を固めてなかったのが問題でしたね。すんません。
| base | 4f745faf7379525299595a2368818a1e3b86c9f6 |
|---|---|
| head | 3de8fa1840b790ad8796c1fac4d750bf430ef969 |
| base branch | jolly/jr-web:develop |
| head branch | jolly/jr-web:battle_ranking_670 |
| status | file |
|---|---|
| modified | lib/public/Ranking-Event.inc |
| modified | public_html/template/Ranking-Event.tpl |