Enhancement(機能追加・改善) #2004
未完了
Yuya Watanabe さんが13年以上前に追加.
12年以上前に更新.
ステータス:
Pending Review(レビュー待ち)
説明
Overview (概要)¶
既存のアクセスブロック機能の下記の点について機能改善を行う.
Spec (仕様)¶
表示
- 登録済みメンバーを一覧表示する
- 各行に下記の内容を表示
- プロフィール画像(PC版のみ)
- ニックネーム
- メンバID
- 削除リンク
- モバイルは10件ずつ,PCは20件ずつ登録メンバーをページャ表示
- 表示順は登録メンバーID順
- その他の表示上の仕様はメンバー検索結果ページの仕様にしたがう
登録
- メンバーIDを入力して登録
- 確認・完了画面は実装しない
- 完了メッセージを表示
- 下記入力値をエラーとする
- 空文字列
- 自然数以外の文字列
- 自分のID
- 存在しないID
- 登録済みのID
- エラー時は値を引き継ぎ,テキストボックス内に表示
削除
- 一覧表示内のリンクを押下することで削除画面確認画面へ遷移
- 確認画面は削除対象メンバーのニックネームとIDを表示する
- 削除完了画面で「削除する」を押下することで実行
- 完了画面は実装しない
- 完了メッセージを表示
- エラーの場合はステータスコード404を表示
ファイル
関連するチケット
1 (0件未完了 — 1件完了)
- ステータス を New(新規) から Accepted(着手) に変更
- ファイル OSS版_アクセスブロック【改善版】仕様.ppt を追加
- ファイル アクセスブロックPC版仕様書.ppt を追加
- ファイル OSS版_アクセスブロック【改善版】仕様.ppt を追加
pull request後の訂正
・日本語の記述を削除
・ページャのサイズがモバイルの場合10であるべきだったものが20であった部分を修正
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
- バグ
- エラーメッセージが仕様書と少し違うように表示される箇所がありました。
- 改善
- 「メンバーID」と「を削除しました。」の間に「PC版半角スペースなし、携帯版半角スペースあり」なので、どちらかに統一した方がいいと思います。
- その他
- 仕様書にCSRF対策の記載がありませんでしたが、下記メッセージが表示されることを確認しました。
[PC] csrf token: 必須項目です。
[携帯] csrf token: 前の画面を読み直して、操作をやり直してください。
詳細につきましては、テスト表で確認お願いします。
仕様書訂正しました。具体的には以下の項目を訂正しました。
- IDという表記をメンバーIDへ全体的に統一する
- 「メンバーID」と「削除しました。」等の間の空白部分がない方に統一する
- モバイルからアクセスブロック一覧画面を閲覧する場合にメンバーIDを表示しない
- ファイル を削除 (
OSS版_アクセスブロック【改善版】仕様.ppt)
- ファイル を削除 (
アクセスブロックPC版仕様書.ppt)
- ファイル を削除 (
OSS版_アクセスブロック【改善版】仕様.ppt)
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
テストOKです。
下記内容が反映されていることを確認しました。
- IDという表記をメンバーIDへ全体的に統一する。
- 「メンバーID」と「削除しました。」等の間の空白部分がない方に統一する。
- モバイルからアクセスブロック一覧画面を閲覧する場合にメンバーIDを表示しない。
- ステータス を Pending Review(レビュー待ち) から Fixed(完了) に変更
- 進捗率 を 50 から 100 に変更
- ステータス を Fixed(完了) から Rejected(差し戻し) に変更
- 進捗率 を 100 から 50 に変更
#1784 完了にしたら本チケットも完了してしまったため、差し戻します。
- ステータス を Rejected(差し戻し) から Accepted(着手) に変更
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 対象バージョン を OpenPNE 3.7.0 から 252 に変更
- 対象バージョン を 252 から OpenPNE 3.8.x に変更
他の形式にエクスポート: Atom
PDF