Review Request 112581: add support for the forum API

[Aaron J. Seigo]

> On Sept. 12, 2013, 3:52 p.m., Aaron J. Seigo wrote:
> > a button that opens a web browser to the forum page makes the application not self-hosting and very clumsy from a navigational POV (how do you get back to the client? pull down the peek area, close the browser and go back to the add-ons client? ugh)
> > 
> > rather, as discussed, it would be better to be able to show the last N (say, 5) comments in the assets info panel. as with reviews, it might make sense to have a link that says "See what people are saying.." or even just "See feedback..." which would then hide that link and expand to show the last N comments.
> > 
> > this itself could be done in one of two ways: either show the comments in the info pane itself or, and this is my preference atm, open a new column showing the feedback. that panel could then have a "Open the forum..." link that would open a web browser and which would be used to interact with the forum.
> > 
> > additionally, on the "rate and review" popup used to add / edit ratings, there should be a text area in which the user can put some text as a review which then gets added to the forum
> > 
> > really, the web forum part is only for the developer to continue interacting with the feedback and for people who wish to get more involved in the conversation to do so. but the Add Ons app itself should provide a basic display the feedback from the forum without leaving the add-ons app.
> 
> Giorgos Tsiapaliokas wrote:
>     > and this is my preference atm, open a new column showing the feedback
>     
>     ok, lets create a new column then.
>     
>     > to be able to show the last N (say, 5) comments 
>     
>     I don't understand what to add in the column. If we have the asset 'Poker 11' there will be
>     a category for the asset 'Poker 11' which will have topics which may have >=0 posts.
>     So, don't we need a column which will show the topics of the asset 'Poker 11' and *another* column
>     which will show the posts (if any) of the specified topic?
>     
>     
>     Also we need some Qt code for the discourse API, where should I put it? In libbodega or in another library?

> I don't understand what to add in the column.

it should show the comments from topics under the asset's category from discourse. it is unnecessary to have multiple columns, imho. for instance, just the first comment in each topic/thread in the 'Poker 11' asset could be shown with the ability to expand and show replies if there are any. what is shown in the Add Ons client does not need to be complex.

> Also we need some Qt code for the discourse API

absolutely not. discourse is an implementation in bodega. anything using bodega should not have to rely on the implementation detail of "bodega uses discourse". anything that a client needs to do with the feedback must be available via a bodega API call.

obviously, bodega does not need to wrap the entire discourse API. all it needs is to be able to fetch comments related to an asset and to post a new (top-level) comment as feedback. everything else more than that can be done via the discourse web interface (replying to threads, reading responses to threads, etc.)


- Aaron J.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112581/#review39893
-----------------------------------------------------------


On Sept. 7, 2013, 5:46 p.m., Giorgos Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112581/
> -----------------------------------------------------------
> 
> (Updated Sept. 7, 2013, 5:46 p.m.)
> 
> 
> Review request for Bodega.
> 
> 
> Description
> -------
> 
> This patch
> * parses the forum info from asset/:assetId
> * adds a button in the ui in order to open the link in the browser
> 
> 
> Diffs
> -----
> 
>   activeclient/package/contents/ui/storebrowser/AssetColumn.qml a678f16 
>   activeclient/src/bodegastore.cpp ba9dc27 
>   lib/bodega/assetjob.cpp 5f539cb 
>   lib/bodega/globals.h 5ab45da 
> 
> Diff: http://git.reviewboard.kde.org/r/112581/diff/
> 
> 
> Testing
> -------
> 
> attached screenshot
> 
> 
> File Attachments
> ----------------
> 
> 
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/09/07/forumbutton.png
> 
> 
> Thanks,
> 
> Giorgos Tsiapaliokas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/active/attachments/20130915/85631a8d/attachment.html>