1112507 - Review Revision page should contain revision comments for all locales

Status: RESOLVED FIXED

(support.mozilla.org :: Knowledge Base Software, task)

Description

[Kadir Topal [:atopal]]

Bug 696658 started showing the revision comment on the review page, but it's only shown for English, we'd need that for localized articles as well. Also, it only shows the comment of the revision being reviewed, we need previous comments as well until the revision marked "current".

Here is how it should look like: http://note.io/1qkVZ9b

Comment 1

[Safwan Rahman (:safwan)]

Topal, So need to be done 2 things
# Show Revision Comment in the Review page of All Locals
# Show Previous Revision Comment

In your Screenshot, I am seeing same 2 comments of same Revision 67751. Can you please provide a simple STR for the 2nd one? So it will be clear!

Comment 2

[Kadir Topal [:atopal]]

Hey Safwan, the text is the same because I was lazy, those would be different revisions.

Comment 3

[Safwan Rahman (:safwan)]

:D Topal, thanks for the Information. So I think according to the Bug Title, this below things need to be done
# Show Previous Revision Comment in the Review page of All Locals

So the STR should be as following:
# Create a Article and approve it.
# Make an Edit/Revision and "dont" approve it.(Regard this Revision number is "1100")
# Make Another Revision and go to Review page of the Revision.(Regard this Revision number is "2244")
Actual Result:
# You can only see the comment of the "Revision: 2244")
Expected Result
# You should also see the comment of "Revision:1100"

Topal, Is the STR right? ;)

And regarding the "#Show Revision Comment in the Review page of All Locals", Can you please open another bug for that? so we can track that on there!

Comment 4

[Tim [:pollti]]

Attachment: add bug 696658's feature for locales

This adds bug 696658's feature for all locales. A feature to show all comments should still be added.

Comment 5

[Tim [:pollti]]

Attachment: add bug 696658's feature for locales

Comment 6

[Michael Cooper [:mythmon]]

I deployed https://github.com/mozilla/kitsune/commit/3703fae461d574658166c8848c9e809d3073c086 to prod.

Comment 7

[Tim [:pollti]]

Thanks, mythmon! However, it's only the first part of this bug. The other revison comments should still be added to the review page.

Comment 8

[Safwan Rahman (:safwan)]

Pollti, I think we should open another bug for tracking that!
I am changing this Bug description and closing it!
Thanks a lot for your contribution

Comment 9

[Safwan Rahman (:safwan)]

Pollti, Here I filled another Bug 1150014.

Comment 10

[Tim [:pollti]]

Thanks, safwan! Maybe I'll find the time to take bug 1150014 in the future.