> On Sept. 8, 2014, 6:22 p.m., Ben Cooksley wrote:
> > What is the status of this? Is there anything blocking it being shipped and 
> > made available on api.kde.org?
> 
> Denis Steckelmacher wrote:
>     It works locally, so I'm just waiting for a ship-it or other comments :-)
> 
> Aleix Pol Gonzalez wrote:
>     Let's see if we can get this discussed in the KF5 BoF today.
> 
> Aleix Pol Gonzalez wrote:
>     I kind of forgot... :(
>     
>     Anyway, +1 for me. If there are no reasons to keep postponing it, I'd say 
> let's just do this.

The only issue I see is that the email notifications has the admin url 
included, which it appears I cannot access, making it hard to find the new 
comment.  There was a suggested change to juvia, which would fix this.  Has 
that been applied?

Otherwise, +1 for me too.


- Matthew


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119991/#review66088
-----------------------------------------------------------


On Aug. 29, 2014, 10:16 a.m., Denis Steckelmacher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119991/
> -----------------------------------------------------------
> 
> (Updated Aug. 29, 2014, 10:16 a.m.)
> 
> 
> Review request for KDE Frameworks and Aurélien Gâteau.
> 
> 
> Repository: kapidox
> 
> 
> Description
> -------
> 
> Juvia is a Free Software commenting system that can easily be used on static 
> websites like api.kde.org. An instance of Juvia has just been installed on 
> commenting.kde.org (many thanks to Ben Cooksley!), and this patch adds 
> support for it to api.kde.org.
> 
> The users can now comment class pages. The comments are disabled (no comment 
> box nor anything else appears) on the main page of each framework, on the 
> Frameworks 5 index page and on any other page that does not directly concern 
> a class. I've done that in order to avoid cluttering important pages with 
> comments, but if you think that having comments on all the pages (or a bigger 
> subset of them) is desirable, it is very easy to change. Personally, I would 
> avoid having comments on the main pages, so that any spam, if the automatic 
> Akismet filter does not work, will not be too visible.
> 
> A screenshot is linked to this review request and shows how the comments have 
> been integrated (I slightly modified the built-in Juvia style so that Doxygen 
> and KDE colors are used).
> 
> 
> Diffs
> -----
> 
>   src/kapidox/data/htmlresource/kde.css e173dfe 
>   src/kapidox/data/templates/comments.html PRE-CREATION 
>   src/kapidox/data/templates/doxygen.html d00e14e 
> 
> Diff: https://git.reviewboard.kde.org/r/119991/diff/
> 
> 
> Testing
> -------
> 
> Posting a comment works, and comments can be viewed. They also appear in the 
> admin interface of Juvia (I've now deleted these comments). I posted comments 
> on different classes and in different frameworks in order to test that 
> namespacing works correctly.
> 
> 
> File Attachments
> ----------------
> 
> Comments on api.kde.org
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2014/08/29/278299d8-18a6-46a0-ada5-1b6452a3276f__apidox-comments-1.png
> 
> 
> Thanks,
> 
> Denis Steckelmacher
> 
>

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to