Re: [kde-doc-english] Policy about RR's for docbook changes

2016-07-29 Thread Luigi Toscano 
Burkhard Lück ha scritto:
> Hi,
> 
> if I file a review request for docbooks I always add two groups/devels as 
> reviewers:
> 
> 1) group docs to check for conformity with docbook rules, wording, markup etc.
> 
> 2) devel group or persons to check the technical aspect + feature correctness 
> in the update.
> 
> Adding two different groups/persons to the RR I expect to get an OK="Ship It" 
> from both groups for the different aspects of the review ;-).
> 
> If I get first a "Ship It" from a docs group member and the RR is only about 
> docbook related issues I usually commit, but not in case the RR is about 
> behavioral/feature changes, then I' need to have a confirmation from a 
> developer.

Right.

> The other way round (first a "Ship It" from a devel group member) I am always 
> unsure how to handle that.
> Do you think there is already a "Ship It" for the RR, no need to add an OK 
> from groups docs and I should just wait x days to commit or should I expect 
> an 
> explicit "Ship It" from the group docs?

I would go and commit; at most. I would expect few days and commit (a "Ship
it" from a developer of the application should mean that the content is 
accurate).

Ciao
-- 
Luigi
___
kde-doc-english mailing list
kde-doc-english@kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english

Re: [kde-doc-english] Review Request 128553: Proofread Okteta Handbook for 16.08

2016-07-29 Thread Friedrich W. H. Kossebau 

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




doc/index.docbook (line 41)


"0.20" is the final version of Okteta as part of Applications 16.08.
(0.20.60, as in master, is the current WIP version of what will be in 16.12)



doc/index.docbook (line 50)


Okteta is part of the kdesdk module these days, so kdesdk might be more 
helpful here (as the first revision of this patch did).


- Friedrich W. H. Kossebau


On July 29, 2016, 4:57 p.m., Burkhard Lück wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128553/
> ---
> 
> (Updated July 29, 2016, 4:57 p.m.)
> 
> 
> Review request for Documentation and Friedrich W. H. Kossebau.
> 
> 
> Repository: okteta
> 
> 
> Description
> ---
> 
> proofread
> remove entities okteta, kappname + package
> bump date + releaseinfo
> resort menuitems
> remove sect appendix
> 
> 
> Diffs
> -
> 
>   doc/index.docbook 8a97c7c 
> 
> Diff: https://git.reviewboard.kde.org/r/128553/diff/
> 
> 
> Testing
> ---
> 
> passes checkXML5
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

___
kde-doc-english mailing list
kde-doc-english@kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english

Re: [kde-doc-english] Review Request 128553: Proofread Okteta Handbook for 16.08

2016-07-29 Thread Burkhard Lück 

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

(Updated Juli 29, 2016, 4:57 nachm.)


Review request for Documentation and Friedrich W. H. Kossebau.


Changes
---

Fix summary + actually bump date + releaseinfo now


Summary (updated)
-

Proofread Okteta Handbook for 16.08


Repository: okteta


Description (updated)
---

proofread
remove entities okteta, kappname + package
bump date + releaseinfo
resort menuitems
remove sect appendix


Diffs (updated)
-

  doc/index.docbook 8a97c7c 

Diff: https://git.reviewboard.kde.org/r/128553/diff/


Testing
---

passes checkXML5


Thanks,

Burkhard Lück

___
kde-doc-english mailing list
kde-doc-english@kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english

[kde-doc-english] [krfb] doc: Fix minor glitches

2016-07-29 Thread Yuri Chornoivan 
Git commit 1b6702167db7ddfe797b089c30e7ae7f7d06ae8c by Yuri Chornoivan.
Committed on 29/07/2016 at 16:45.
Pushed by yurchor into branch 'master'.

Fix minor glitches

M  +3-3doc/index.docbook

http://commits.kde.org/krfb/1b6702167db7ddfe797b089c30e7ae7f7d06ae8c

diff --git a/doc/index.docbook b/doc/index.docbook
index 1484761..7325663 100644
--- a/doc/index.docbook
+++ b/doc/index.docbook
@@ -163,14 +163,14 @@ The address is just a hint - you can use any address that 
can
 reach your computer.  tries to guess your address from your
 network configuration, but does not always succeed in doing so.
 If your computer is behind a firewall it may have a different
-address or be unreachable for other computers
+address or be unreachable for other computers.
 
 
 You can change the port on the Network
 page in the configuration dialog.
 
 
-The next field is prefilled with an automatical generated password.
+The next field is prefilled with an automatically generated password.
 Click in the icon at the right of the field to change the password.
 
 
@@ -296,7 +296,7 @@ the correct port.
 
 
 The Security page allows you to configure whether the
-person connecting to the  servercan control the desktop, or only view.
+person connecting to the  server can control the desktop, or only view.
 
 
 
___
kde-doc-english mailing list
kde-doc-english@kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english

Re: [kde-doc-english] Review Request 128547: Update Akregator docbook to 16.08

2016-07-29 Thread Burkhard Lück 

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

(Updated Juli 29, 2016, 4:25 nachm.)


Review request for Documentation, KDEPIM and Laurent Montel.


Changes
---

Replace removed Share Services with URL Interceptor page in settings dialog


Repository: kdepim


Description
---

proofread + update
new screenshots not uploaded
bump date + releaseinfo
remove unused entities kappname + package and akregator already in kdoctools
remove appendix installation

@Laurent:
the feature Share Services no longer in kf5?
Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?


Diffs (updated)
-

  doc/akregator/index.docbook f34a8ce 

Diff: https://git.reviewboard.kde.org/r/128547/diff/


Testing
---

passes checkXML5


Thanks,

Burkhard Lück

___
kde-doc-english mailing list
kde-doc-english@kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english

Re: [kde-doc-english] Review Request 128547: Update Akregator docbook to 16.08

2016-07-29 Thread Laurent Montel 


> On July 29, 2016, 11:39 a.m., Laurent Montel wrote:
> > "the feature Share Services no longer in kf5?" yep removed in kf5 
> > 
> > "Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?" 
> > nope it's specific to akregator/kmail I reimplemented it to support 
> > qtwebengine. So it's not the same.
> 
> Burkhard Lück wrote:
> >> Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?
> > nope it's specific to akregator/kmail I reimplemented it to support 
> qtwebengine. So it's not the same.
> 
> It is not the same for you from devel point of view, true.
> But for joe user it looks very similar to konquerors AdBlock page and it 
> seems to have similar features, 
> please look at 
> https://docs.kde.org/trunk5/en/kde-workspace/kcontrol/khtml-adblock/index.html
> 
> Thanks

Oh yes indeed for enduser it's the same :)


- Laurent


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


On July 29, 2016, 8:17 a.m., Burkhard Lück wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128547/
> ---
> 
> (Updated July 29, 2016, 8:17 a.m.)
> 
> 
> Review request for Documentation, KDEPIM and Laurent Montel.
> 
> 
> Repository: kdepim
> 
> 
> Description
> ---
> 
> proofread + update
> new screenshots not uploaded
> bump date + releaseinfo
> remove unused entities kappname + package and akregator already in kdoctools
> remove appendix installation
> 
> @Laurent:
> the feature Share Services no longer in kf5?
> Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?
> 
> 
> Diffs
> -
> 
>   doc/akregator/index.docbook f34a8ce 
> 
> Diff: https://git.reviewboard.kde.org/r/128547/diff/
> 
> 
> Testing
> ---
> 
> passes checkXML5
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

___
kde-doc-english mailing list
kde-doc-english@kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english

Re: [kde-doc-english] Review Request 128547: Update Akregator docbook to 16.08

2016-07-29 Thread Burkhard Lück 


> On Juli 29, 2016, 11:39 vorm., Laurent Montel wrote:
> > "the feature Share Services no longer in kf5?" yep removed in kf5 
> > 
> > "Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?" 
> > nope it's specific to akregator/kmail I reimplemented it to support 
> > qtwebengine. So it's not the same.

>> Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?
> nope it's specific to akregator/kmail I reimplemented it to support 
> qtwebengine. So it's not the same.

It is not the same for you from devel point of view, true.
But for joe user it looks very similar to konquerors AdBlock page and it seems 
to have similar features, 
please look at 
https://docs.kde.org/trunk5/en/kde-workspace/kcontrol/khtml-adblock/index.html

Thanks


- Burkhard


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


On Juli 29, 2016, 8:17 vorm., Burkhard Lück wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128547/
> ---
> 
> (Updated Juli 29, 2016, 8:17 vorm.)
> 
> 
> Review request for Documentation, KDEPIM and Laurent Montel.
> 
> 
> Repository: kdepim
> 
> 
> Description
> ---
> 
> proofread + update
> new screenshots not uploaded
> bump date + releaseinfo
> remove unused entities kappname + package and akregator already in kdoctools
> remove appendix installation
> 
> @Laurent:
> the feature Share Services no longer in kf5?
> Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?
> 
> 
> Diffs
> -
> 
>   doc/akregator/index.docbook f34a8ce 
> 
> Diff: https://git.reviewboard.kde.org/r/128547/diff/
> 
> 
> Testing
> ---
> 
> passes checkXML5
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

___
kde-doc-english mailing list
kde-doc-english@kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english

Re: [kde-doc-english] Review Request 128547: Update Akregator docbook to 16.08

2016-07-29 Thread Laurent Montel 

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



"the feature Share Services no longer in kf5?" yep removed in kf5 

"Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?" nope 
it's specific to akregator/kmail I reimplemented it to support qtwebengine. So 
it's not the same.

- Laurent Montel


On July 29, 2016, 8:17 a.m., Burkhard Lück wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128547/
> ---
> 
> (Updated July 29, 2016, 8:17 a.m.)
> 
> 
> Review request for Documentation, KDEPIM and Laurent Montel.
> 
> 
> Repository: kdepim
> 
> 
> Description
> ---
> 
> proofread + update
> new screenshots not uploaded
> bump date + releaseinfo
> remove unused entities kappname + package and akregator already in kdoctools
> remove appendix installation
> 
> @Laurent:
> the feature Share Services no longer in kf5?
> Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?
> 
> 
> Diffs
> -
> 
>   doc/akregator/index.docbook f34a8ce 
> 
> Diff: https://git.reviewboard.kde.org/r/128547/diff/
> 
> 
> Testing
> ---
> 
> passes checkXML5
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

___
kde-doc-english mailing list
kde-doc-english@kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english

[kde-doc-english] Review Request 128547: Update Akregator docbook to 16.08

2016-07-29 Thread Burkhard Lück 

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

Review request for Documentation, KDEPIM and Laurent Montel.


Repository: kdepim


Description
---

proofread + update
new screenshots not uploaded
bump date + releaseinfo
remove unused entities kappname + package and akregator already in kdoctools
remove appendix installation

@Laurent:
the feature Share Services no longer in kf5?
Web Engine URL Interceptor Plugins page = AdBlock page from konqueror?


Diffs
-

  doc/akregator/index.docbook f34a8ce 

Diff: https://git.reviewboard.kde.org/r/128547/diff/


Testing
---

passes checkXML5


Thanks,

Burkhard Lück

___
kde-doc-english mailing list
kde-doc-english@kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english