> On Dec. 3, 2016, 10:19 p.m., Daniel Vrátil wrote: > > Looks good to me > > Burkhard Lück wrote: > What about my question: > "There is not much content in this docbook and parts of these options are > already in the kmail handbook. > Maybe we should move everything into the kmail/kontact docbooks are and > delete this docbook?" > > Luigi Toscano wrote: > The changes themselves look good. IMHO (and waiting on developers), the > options for specific programs should be documented in the respective > documentation, and then there would probably be no need for this manual (even > if it was something left, it could go into the main kontact manual).
Yup, makes sense to merge into kontact docbook. - Daniel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129561/#review101254 ----------------------------------------------------------- On Nov. 26, 2016, 2:13 p.m., Burkhard Lück wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129561/ > ----------------------------------------------------------- > > (Updated Nov. 26, 2016, 2:13 p.m.) > > > Review request for Documentation, KDEPIM and Allen Winter. > > > Repository: kontact > > > Description > ------- > > remove kappname + packages, no longer used > bump date + releaseinfo > remove "Outlook compatible invitation reply comments", I could not find it in > the code > remove empty chapter Common Problems > remove appendix > > There is not much content in this docbook and parts of these options are > already in the kmail handbook. > Maybe we should move everything into the kmail/kontact docbooks are and > delete this docbook? > Any comments? > > > Diffs > ----- > > doc/kontact-admin/index.docbook dd21a2c > > Diff: https://git.reviewboard.kde.org/r/129561/diff/ > > > Testing > ------- > > passes checkXML5 > > > Thanks, > > Burkhard Lück > >