D28834: Add metadata properties to calendar

Volker Krause noreply at phabricator.kde.org
Tue Apr 14 20:54:20 BST 2020


vkrause added a comment.


  In D28834#648405 <https://phabricator.kde.org/D28834#648405>, @winterz wrote:
  
  > I don't know how things are done in frameworks but it seems to me that the KF5_VERSION (see top of kcalendarcore/CMakeLists.txt) needs to become 5.70.0 now
  
  
  This is handled automatically, no need to change this.

INLINE COMMENTS

> calendar.h:101
> +*/
> +enum CalendarType
> +{

As already noted in the previous review, "type" isn't the best naming for something that is about access control/permission IMHO.

A possible alternative name could be something like "AccessMode", or maybe even just bool isReadOnly.

REPOSITORY
  R172 KCalendar Core

BRANCH
  props

REVISION DETAIL
  https://phabricator.kde.org/D28834

To: nicolasfella, #frameworks, #kde_pim, vkrause, winterz
Cc: winterz, kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200414/b1f440b6/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list