Project

General

Profile

Bug（バグ） #3303

通知センターのUIがモックアップ上のものと異なる箇所がある

Added by Youichi Kimura over 10 years ago. Updated over 3 years ago.

Status:
Fixed（完了）
Priority:
Normal（通常）
Assignee:
isao sano
Target version:
OpenPNE 3.10.0
Start date:
2015-11-09
Due date:
% Done:

100%

3.6 で発生するか:
No (いいえ)
3.8 で発生するか:
Unknown (未調査)

Description

Overview (現象)

http://www.tejimaya.com/20111203houou-%E3%83%A2%E3%83%83%E3%82%AF%E3%82%A2%E3%83%83%E3%83%97-update/

http://www.tejimaya.com/20111203%E9%80%9A%E7%9F%A5%E3%82%BB%E3%83%B3%E3%82%BF%E3%83%BC%E3%82%92%E5%AE%9F%E8%A3%85%E3%81%99%E3%82%8B%E3%82%A2%E3%82%A4%E3%83%87%E3%82%A2/

通知センターのモックアップが上記のURLに掲載されているが、次の点で現在の実装と相違がある。

  • フレンド申請の通知文が「○○さんからリンク申請が届いています」ではなく「フレンド申請を許可しますか？」のみとなっている（申請者の名前が表示されていない）
  • 「クリックしたものは通知から消す」との記述があるが、既読状態にかかわらず全ての通知が表示されている

Way to fix (修正内容)

モックアップの通りに修正すべきか判断し、モックアップに合わせる場合は上記の相違点を修正する。

スクリーンショット_2016-08-30_13.13.42.png View (65.6 KB) Youichi Kimura, 2016-08-30 13:14

Subtasks

Backport（バックポート） #3860: 通知センターのUIがモックアップ上のものと異なる箇所があるFixed（完了）kaoru n

Associated revisions

Revision 21d3da66 (diff)
Added by isao sano over 6 years ago

Modify notification message of friend-link application (refs #3303)

notification message body is not used in template.
it can be changed without untoword results.

Revision 854f81ad (diff)
Added by isao sano over 6 years ago

Modify notification center header template (refs #3303)

Revision 7438ab04 (diff)
Added by isao sano over 6 years ago

Add delete method to opNotificationCenter class (refs #3303)

Revision cf25269d (diff)
Added by isao sano over 6 years ago

Delete notification when friend link application is accepted. (refs #3303)

Revision 2604a23c (diff)
Added by isao sano over 6 years ago

Change to fix to add helper (refs #3303)

Revision 7d5c381d (diff)
Added by isao sano over 6 years ago

Adjust indentation (refs #3303)

Revision bd34075a (diff)
Added by isao sano almost 5 years ago

Delete unnecessary characters (refs #3303)

Revision 3102758e (diff)
Added by isao sano almost 5 years ago

Correct the part that does not require branching (refs #3303)

Revision c130e37f (diff)
Added by isao sano almost 5 years ago

Delete unnecessary processing (refs #3303)

History

#1 Updated by 誠二 天重 about 9 years ago

  • Status changed from New（新規） to Pending Review（レビュー待ち）
  • Assignee set to 誠二 天重
  • % Done changed from 0 to 50
  • 3.6 で発生するか changed from Unknown (未調査) to No (いいえ)

https://github.com/openpne/OpenPNE3/pull/175
にてプルリクエスト。
文言としては

ja フレンド申請 （申請者名）
en Friend request from （申請者名）

とし、メンバーのプロフィール画面へのリンクを貼っている。
また、メンバー申請を受理または拒否した場合に、該当するお知らせを削除するよう修正。

#2 Updated by kaoru n over 8 years ago

  • Assignee deleted (誠二 天重)

#3 Updated by isao sano almost 8 years ago

#4 Updated by kaoru n about 7 years ago

  • Assignee set to isao sano

#5 Updated by Youichi Kimura about 7 years ago

  • 修正前の時点で未承認のフレンド申請が存在していた場合、修正後にその申請の通知を表示すると本来ニックネームが表示される部分に「Do you accept friend request?」と表示されます
  • lib/model/doctrine/MemberRelationship.class.php に不要な , が存在しているため削除して下さい

#6 Updated by isao sano over 6 years ago

  • Status changed from Rejected（差し戻し） to Accepted（着手）
  • % Done changed from 50 to 0

#7 Updated by isao sano over 6 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

note5 に関して修正をし、masterブランチに対して新たにプルリクエストを行いました。
https://github.com/openpne/OpenPNE3/pull/460

#8 Updated by isao sano over 6 years ago

  • Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#9 Updated by isao sano over 6 years ago

  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

#10 Updated by isao sano over 5 years ago

  • Status changed from Rejected（差し戻し） to Accepted（着手）
  • % Done changed from 50 to 0

#11 Updated by isao sano over 5 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

新しい master ブランチ向けにプルリクエストしました。
レビューお願いします。

https://github.com/openpne/OpenPNE3/pull/460

#12 Updated by Rimpei Ogawa about 5 years ago

  • Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）

#13 Updated by isao sano almost 5 years ago

  • Status changed from Rejected（差し戻し） to Accepted（着手）
  • % Done changed from 50 to 0

#14 Updated by isao sano almost 5 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

修正して、再度プルリクエストを行いました。
https://github.com/openpne/OpenPNE3/pull/460

#15 Updated by kaoru n over 3 years ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE 3.10.x

#16 Updated by Rimpei Ogawa over 3 years ago

  • Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
  • % Done changed from 50 to 70

#17 Updated by kaoru n over 3 years ago

  • Target version changed from OpenPNE 3.10.x to OpenPNE 3.9.8

#18 Updated by kaoru n over 3 years ago

  • Target version changed from OpenPNE 3.9.8 to OpenPNE 3.10.0

#19 Updated by isao sano over 3 years ago

  • Status changed from Pending Testing（テスト待ち） to Pending Merge（マージ待ち）
  • % Done changed from 70 to 80

試験完了です。問題ありません。

#20 Updated by kaoru n over 3 years ago

  • Status changed from Pending Merge（マージ待ち） to Fixed（完了）
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF