<div dir="ltr">Thanks Daisy,<div><br></div><div>You are correct.</div><div><br></div><div>I rechecked both master and stable-liberty translations more carefully</div><div>and it turned out it is a corner case.</div><div>I noticed all of such strings are rejected in both master and stable-liberty<br></div><div>and most of them (all? I am not sure) are edited through "Project-wide Search & Replace".</div><div>When we edit "rejected" strings with "Project-wide Search & Replace",</div><div>the strings still remains in "rejected" status.</div><div><br></div><div>It seems that I started to review nova translations too early and<br></div><div>unfortunately we cannot complete reviews due to the volume.</div><div>As a result, we hit "not a good practice" you said.</div><div><br></div><div>I learned one practice that if we are not confident with addressing "rejected" strings</div><div>which already exist in master branch, we should not touch such string,</div><div>I learned much more detail conditions, but perhaps it only applies to me.</div><div><br></div><div>Thanks,</div><div>Akihiro</div><div><br></div><div><div class="gmail_extra"><div class="gmail_quote">2015-10-08 22:39 GMT+09:00 Ying Chun Guo <span dir="ltr"><<a href="mailto:guoyingc@cn.ibm.com" target="_blank">guoyingc@cn.ibm.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><font size="2" face="sans-serif">I cannot agree with you, Akihiro.</font>
<br><font size="2" face="sans-serif">When stable-liberty version was created,
we set master version as "Read Only".</font>
<br><font size="2" face="sans-serif">Translators had to improve stable-liberty
branch.</font>
<br><font size="2" face="sans-serif">The corrections made in stable-liberty
version is newer than the old translations in master version.</font>
<br><font size="2" face="sans-serif">So when I merged, the newer translations
will be copied and replaced the old translations.</font>
<br><font size="2" face="sans-serif">So the corrections will be moved to
the master version.</font>
<br>
<br><font size="2" face="sans-serif">But the rejections will not.</font>
<br><font size="2" face="sans-serif">So it's not a good practise to only
reject translations in stable-liberty without input the correct translations.</font>
<br>
<br><font size="2" face="sans-serif">Could you check in Nova master version?</font>
<br><font size="2" face="sans-serif">I think your corrections to IBM translations
should be moved to master version.</font>
<br><span class="">
<br><font size="2" face="sans-serif">Best regards<br>
Ying Chun Guo (Daisy)<br>
</font>
<br>
<br></span><tt><font size="2">Akihiro Motoki <<a href="mailto:amotoki@gmail.com" target="_blank">amotoki@gmail.com</a>> wrote on
10/08/2015 09:31:36 PM:<br>
<br>
> From: Akihiro Motoki <<a href="mailto:amotoki@gmail.com" target="_blank">amotoki@gmail.com</a>></font></tt>
<br><tt><font size="2">> To: Ying Chun Guo/China/IBM@IBMCN</font></tt>
<br><tt><font size="2">> Cc: Openstack-i18n <<a href="mailto:openstack-i18n@lists.openstack.org" target="_blank">openstack-i18n@lists.openstack.org</a>></font></tt>
<br><tt><font size="2">> Date: 10/08/2015 09:32 PM</font></tt>
<br><tt><font size="2">> Subject: Re: [Openstack-i18n] Approved &
rejected translations after<br>
> versions merging</font></tt>
<br><div class=""><div class="h5"><tt><font size="2">> <br>
> On a second thought, I start to think this can happen in every <br>
> OpenStack release.</font></tt>
<br><tt><font size="2">> <br>
> What can possibly happen?</font></tt>
<br><tt><font size="2">> <br>
> Most reviews are done after RC1 is released. It means reviews and
<br>
> corrections are done in stable-xxxx branch.</font></tt>
<br><tt><font size="2">> It is not a rare case that strings are also translated
in the master branch.</font></tt>
<br><tt><font size="2">> In this case, corrections in stable-xxxx branch
will not be <br>
> feedback-ed to the master branch</font></tt>
<br><tt><font size="2">> and this means that translators/reviewers need
to do the same thing <br>
> again and can lead to</font></tt>
<br><tt><font size="2">> inconsistent translations.</font></tt>
<br><tt><font size="2">> <br>
> What can we do?</font></tt>
<br><tt><font size="2">> <br>
> I don't think we have a general solution to this.</font></tt>
<br><tt><font size="2">> Zanata's current behavior may work well in most
cases, but does not <br>
> work for us.</font></tt>
<br><tt><font size="2">> <br>
> One possible solution is to allow language coordinators (or <br>
> reviewers) to upload PO files.</font></tt>
<br><tt><font size="2">> Language coordinators (or reviewers) can use
diff tools locally and <br>
> merge translations</font></tt>
<br><tt><font size="2">> more efficiently. The demand may be different
among languages.</font></tt>
<br><tt><font size="2">> If they can upload PO files, we can cope with
this lang by lang.</font></tt>
<br><tt><font size="2">> <br>
> Thought?</font></tt>
<br><tt><font size="2">> <br>
> Akihiro</font></tt>
<br><tt><font size="2">> <br>
> 2015-10-08 19:37 GMT+09:00 Akihiro Motoki <<a href="mailto:amotoki@gmail.com" target="_blank">amotoki@gmail.com</a>>:</font></tt>
<br><tt><font size="2">> Daisy,</font></tt>
<br><tt><font size="2">> <br>
> Is there any way to apply modifications made in stable-liberty branch</font></tt>
<br><tt><font size="2">> to the master branch?</font></tt>
<br><tt><font size="2">> <br>
> We made a lot of modifications/fixes to strings from IBM translation<br>
> contribution</font></tt>
<br><tt><font size="2">> in stable-liberty branch. The master branch already
has translations because</font></tt>
<br><tt><font size="2">> it is contributed by IBM, so merging from liberty
to master does not help us.</font></tt>
<br><tt><font size="2">> <br>
> It seems translators or language coordinator cannot upload PO files,</font></tt>
<br><tt><font size="2">> so I cannot have a way to replace the master
translation with the <br>
> liberty version</font></tt>
<br><tt><font size="2">> in a batch way.</font></tt>
<br><tt><font size="2">> <br>
> We don't want to check >100 rejected strings and copy >100 modified
strings </font></tt>
<br><tt><font size="2">> from liberty manually.</font></tt>
<br><tt><font size="2">> <br>
> Is there any suggestion?</font></tt>
<br><tt><font size="2">> <br>
> Akihiro</font></tt>
<br><tt><font size="2">> <br>
> 2015-10-03 2:12 GMT+09:00 Ying Chun Guo <<a href="mailto:guoyingc@cn.ibm.com" target="_blank">guoyingc@cn.ibm.com</a>>:</font></tt>
<br><tt><font size="2">> Hi, <br>
> <br>
> A translation could be approved or rejected. This action is called
<br>
> translation review. <br>
> A version merging is to copy all matching translated/approved <br>
> translations from the source version to the target version. <br>
> If there is an existing translated/approved translation, the newer
<br>
> translation will be used. <br>
> After liberty translations are closed, we will merge translations
<br>
> from stable-liberty version to master version, <br>
> and then open both stable-liberty version and master version to <br>
> accept translations. <br>
> I investigated whether translation review would be copied after <br>
> version merging. <br>
> <br>
> Here are the results: <br>
> The review result - "approve" will be copied to the target
version, <br>
> while the review result - "reject" will not. <br>
> Only "rejecting" existing translations will not have the
same <br>
> translations rejected in master version. <br>
> But "approving" existing translations will have the same
<br>
> translations approved in the master version. <br>
> That means, if there are "rejected" translations in stable-liberty,
<br>
> it's better to input the correct translations. <br>
> <br>
> The investigation result is OK with me. <br>
> If you have different opinions, please propose here. <br>
> If you want to understand more about version merging, please let me
know. <br>
> <br>
> Best regards<br>
> Ying Chun Guo (Daisy)<br>
</font></tt>
<br><tt><font size="2">> _______________________________________________<br>
> Openstack-i18n mailing list<br>
> <a href="mailto:Openstack-i18n@lists.openstack.org" target="_blank">Openstack-i18n@lists.openstack.org</a><br>
> </font></tt><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-i18n" target="_blank"><tt><font size="2">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-i18n</font></tt></a><tt><font size="2"><br>
</font></tt></div></div></blockquote></div><br></div></div></div>