Project

General

Profile

Bug(バグ) #1776

インフォメーションに表示される新着コメントお知らせが赤字になっていない [仕様バグ]

Added by Kiwa Sakai over 8 years ago. Updated about 8 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
Target version:
Start date:
2010-11-10
Due date:
% Done:

100%

3.6 で発生するか:
Yes
[QA]バグ通知済:
Yes
3.8 で発生するか:
Unknown (未調査)

Description

PC,携帯 共に、マイホームのインフォメーション欄に表示される新着コメントのお知らせが赤字にならず「n件の日記に新着コメントがあります!」全てにリンクが貼られている。
OpenPNEの新着お知らせは「赤字インフォメーション リンク」という表示が一般的なので、合わせるべき。

pc_frontend-member-home-diary_information.png View - 新着コメントお知らせのUIイメージ(PC) (5.92 KB) Kiwa Sakai, 2010-11-10 21:26

mobile_frontend-member-home-diary_information.png View - 新着コメントお知らせのUIイメージ(携帯) (7.72 KB) Kiwa Sakai, 2010-11-10 21:26


Related issues

Related to opDiaryPlugin - Backport(バックポート) #2802: インフォメーションに表示される新着コメントお知らせが赤字になっていない [仕様バグ] Fixed(完了) 2010-11-10

Associated revisions

Revision 32ff23fa (diff)
Added by Yuya Watanabe about 8 years ago

give color to notice messages (refs #1776)

Revision 6967f278 (diff)
Added by Yuya Watanabe about 8 years ago

fixxed wrong displayed word (fixed #1776)

Revision 59edab57 (diff)
Added by Yuya Watanabe about 8 years ago

replace filename 'unread' to 'caution' (fixed #1776)

Revision de6550fc (diff)
Added by Yuya Watanabe about 8 years ago

remove unnecessary character (fixed #1776)

Revision bb09adda
Added by Rimpei Ogawa about 8 years ago

Merge pull request #5 from 'nise-nabe/notify-color'

give color to notice messages (refs #1776)

Conflicts:
apps/mobile_frontend/i18n/messages.ja.xml

History

#1 Updated by Kousuke Ebihara over 8 years ago

  • Priority changed from Normal(通常) to High(高め)

#2 Updated by Kousuke Ebihara over 8 years ago

  • [QA]バグ通知済 set to No

#3 Updated by Kousuke Ebihara over 8 years ago

  • [QA]バグ通知済 changed from No to Yes

#4 Updated by Rimpei Ogawa over 8 years ago

  • Target version set to v1.3.2

#5 Updated by Shingo Yamada about 8 years ago

  • Assignee set to Rimpei Ogawa

#6 Updated by Yuya Watanabe about 8 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • Assignee changed from Rimpei Ogawa to Yuya Watanabe
  • % Done changed from 0 to 50

#7 Updated by Naoya Tozuka about 8 years ago

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

修正点確認しました。OKです。

#8 Updated by Rimpei Ogawa about 8 years ago

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

https://github.com/balibali/opDiaryPlugin/pull/5
でコメントしています。

#9 Updated by Yuya Watanabe about 8 years ago

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

コメントの内容を反映した修正を行いました。
https://github.com/nise-nabe/opDiaryPlugin/compare/master...de6550fc

コメント内容のほか、通知部分で「日記を読む」となっていた部分を「コメントを読む」という文言に変更しました。

#10 Updated by Anonymous about 8 years ago

更新履歴 de6550fc020cb46b11e9fb19c2299691d3f2a646 で適用されました。

#11 Updated by Anonymous about 8 years ago

更新履歴 59edab578ee15aa1621a78e79b313c24d8a28a5f で適用されました。

#12 Updated by Anonymous about 8 years ago

更新履歴 6967f2786d6054674cf9401cc0320ea5a9b3afd1 で適用されました。

#13 Updated by Naoya Tozuka about 8 years ago

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

https://github.com/balibali/opDiaryPlugin/pull/5 のコメントも踏まえ適切な修正が行われているのを確認しました。(6967f27, 59edab5, de6550f)

レビューOKです。

#14 Updated by Rimpei Ogawa about 8 years ago

テストを実施してもらえるようなので master ブランチに変更をマージしました。

#15 Updated by isao sano about 8 years ago

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

このチケットで扱う問題ではないと感じますが、テスト中にバグを発見したので差し戻します。

1. Aでログイン後、日記作成する。
2. Bでログイン後、Aの日記にコメントする。
3. コメント投稿後、そのコメントを削除する。
4. ログアウトする
5. Aでログイン後、マイホームのインフォメーション欄を確認する。

この手順を行った場合、インフォメーション欄に
「1 件の日記に新着コメントがあります」と表示されます。

#16 Updated by isao sano about 8 years ago

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

http://redmine.openpne.jp/issues/1776#note-15
の件はこのチケットでは対応しません。
また、プラグイン実装者によると仕様ということでしたが、仕様バグという形で以下の別チケットでの対応とします。

Bug(バグ) #2282
(仕様バグ)日記コメントを削除しても、インフォメーション欄に新着コメントお知らせが残る
http://redmine.openpne.jp/issues/2282

上記の理由から差し戻す理由がないため、元のテスト待ちの状態にステータスを変更いたします。

#17 Updated by isao sano about 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テスト完了いたしました。
問題ありません。このちけっとをFixedいたします。

Also available in: Atom PDF