Project

General

Profile

Backport（バックポート） #3860

Bug（バグ） #3303: 通知センターのUIがモックアップ上のものと異なる箇所がある

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

Added by isao sano about 8 years ago. Updated over 3 years ago.

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

100%

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 (修正内容)

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

Related issues

Related to opSkinThemePlugin - Bug（バグ） #4419: #3860 の修正を opSkinThemePlugin にも適用する Fixed（完了） 2020-03-19

Associated revisions

Revision ccd096ab (diff)
Added by isao sano about 8 years ago

Modify notification message of friend-link application (refs #3860, BP from #3303)

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

Revision 28fc1d0a (diff)
Added by isao sano about 8 years ago

Modify notification center header template (refs #3860, BP from #3303)

Revision beb5f964 (diff)
Added by isao sano about 8 years ago

Add delete method to opNotificationCenter class (refs #3860, BP from #3303)

Revision 1f314513 (diff)
Added by isao sano about 8 years ago

Delete notification when friend link application is accepted. (refs #3860, BP from #3303)

Revision 13c90bc2 (diff)
Added by isao sano almost 6 years ago

Change to fix to add helper (refs #3860, BP from #3303)

Revision f393ad3f (diff)
Added by isao sano almost 6 years ago

Adjust indentation (refs #3860, BP from #3303)

Revision 23b0e2f8 (diff)
Added by isao sano over 3 years ago

Delete unnecessary characters (refs #3860, BP from #3303)

Revision cf194847 (diff)
Added by isao sano over 3 years ago

Correct the part that does not require branching (refs #3860, BP from #3303)

Revision c023bcf8 (diff)
Added by isao sano over 3 years ago

Delete unnecessary processing (refs #3860, BP from #3303)

History

#1 Updated by isao sano about 8 years ago

  • Related to Bug（バグ） #3303: 通知センターのUIがモックアップ上のものと異なる箇所がある added

#2 Updated by isao sano about 8 years ago

  • Status changed from New（新規） to Accepted（着手）

#3 Updated by isao sano about 8 years ago

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

https://github.com/openpne/OpenPNE3/pull/270
にてプルリクエストを行いました。

レビューお願いします。

#4 Updated by kaoru n almost 8 years ago

  • Target version changed from OpenPNE 3.8.18 to OpenPNE 3.8.x

#5 Updated by kaoru n almost 8 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.19

#6 Updated by kaoru n over 7 years ago

  • Target version changed from OpenPNE 3.8.19 to OpenPNE 3.8.x

#7 Updated by kaoru n over 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.21

#8 Updated by Youichi Kimura over 7 years ago

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

#3303-5 と同様の理由で差し戻します。

#9 Updated by kaoru n about 7 years ago

  • Target version changed from OpenPNE 3.8.21 to OpenPNE 3.8.x

#10 Updated by kaoru n about 6 years ago

  • Parent task set to #3303

#11 Updated by isao sano almost 6 years ago

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

#12 Updated by isao sano almost 6 years ago

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

https://github.com/openpne/OpenPNE3/pull/270
修正しました。
レビューお願いします。

#13 Updated by Rimpei Ogawa over 3 years ago

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

こちら親チケットの修正が一部反映されていないようでした。

#14 Updated by kaoru n over 3 years ago

  • Assignee changed from isao sano to kaoru n

https://github.com/openpne/OpenPNE3/pull/579
にてプルリクエストしました

https://github.com/openpne/OpenPNE3/pull/270
についてはcloseします

#15 Updated by Rimpei Ogawa over 3 years ago

  • Status changed from Rejected（差し戻し） to Pending Testing（テスト待ち）
  • % Done changed from 50 to 70

#16 Updated by kaoru n over 3 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.35

#17 Updated by isao sano over 3 years ago

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

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

#18 Updated by kaoru n over 3 years ago

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

マージしました

#19 Updated by kaoru n over 3 years ago

Also available in: Atom PDF