[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-active
Subject:    Re: Discourse categories and active-client
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2013-09-17 10:50:43
Message-ID: 2552560.VMIR9Ivmnu () freedom
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Tuesday, September 17, 2013 12:36:58 Giorgos Tsiapaliokas wrote:
> I think that the best place for DiscourseCategoryJob is the AssetOperations
> class and

agreed.

> of libbodega would be able to take the info from a method like
> AssetOperations::discourseCategory()

a model should be enough.

> and for the QML users we will add a
> AssetOperations::discourseCategoryModel().

a model makes sense. but do not use "discourse" anywhere in the API. use 
"forum" or another neutral word.

i keep saying that bodega using discourse is an implementation detail that 
should not leak to users of bodega. and i mean it. just think: 

* what happens if we replace discourse with something else in the future? 
* why should the client ever even care that it is discourse running the 
forums?

-- 
Aaron J. Seigo
["signature.asc" (application/pgp-signature)]

_______________________________________________
Active mailing list
Active@kde.org
https://mail.kde.org/mailman/listinfo/active


[prev in list] [next in list] [prev in thread] [next in thread]