Merge lp:~libqtelegram-team/telegram-app/converged into lp:telegram-app
| Status: | Merged |
|---|---|
| Approved by: | Jin on 2016-03-14 |
| Approved revision: | 158 |
| Merged at revision: | 163 |
| Proposed branch: | lp:~libqtelegram-team/telegram-app/converged |
| Merge into: | lp:telegram-app |
| Diff against target: |
1014 lines (+258/-209) 22 files modified
telegram/app/asemantools/qml/AsemanTools/Button.qml (+0/-1) telegram/app/qml/AccountAddContactHeader.qml (+1/-1) telegram/app/qml/AccountContactsPage.qml (+5/-5) telegram/app/qml/AccountDialogList.qml (+3/-0) telegram/app/qml/AccountDialogPage.qml (+3/-6) telegram/app/qml/AccountListItem.qml (+4/-10) telegram/app/qml/AccountLoading.qml (+0/-3) telegram/app/qml/AccountMessageList.qml (+4/-4) telegram/app/qml/AccountNamePage.qml (+1/-1) telegram/app/qml/AccountPage.qml (+19/-21) telegram/app/qml/AccountSendMessage.qml (+0/-2) telegram/app/qml/AccountSettings.qml (+6/-6) telegram/app/qml/AccountUsernamePage.qml (+1/-1) telegram/app/qml/AddContactPage.qml (+2/-2) telegram/app/qml/IntroPage.qml (+156/-121) telegram/app/qml/PickerPage.qml (+2/-2) telegram/app/qml/PreviewPage.qml (+4/-4) telegram/app/qml/ProfilePage.qml (+4/-5) telegram/app/qml/components/DialogsListItem.qml (+3/-1) telegram/app/qml/components/TelegramContactsListItem.qml (+1/-1) telegram/app/qml/telegram.qml (+39/-10) telegram/app/telegram.desktop (+0/-2) |
| To merge this branch: | bzr merge lp:~libqtelegram-team/telegram-app/converged |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Roberto Mier Escandón | 2016-02-23 | Approve on 2016-03-11 | |
|
Review via email:
|
|||
| Jin (jindallo) wrote : | # |
Hello Roberto,
Yes, since Karni and I folk this branch from 2.0.5.0 to develop this feature,
the conflicts need to be resolved is expected..
And for the huge amount of modified lines,
we don't have modification on .po files actually,
I will bzr ignore that so that make this clean.
| Jin (jindallo) wrote : | # |
Hello Roberto,
After checked,
the .po files need an update from karni -> jin,
in the path I think that will be acceptable,
since I will build out the click package for releasing,
from main trunk so this make sense to me.
What do you think?
And for the version indicated files: version.
I will bump that after merge this branch into trunk.
For MessageReplyIte
there is one conflict from the modification by Michael with trunk:r158,
I will resolve this one later, thanks!
For en_GB.po, I will check this as well,
since I always rebuild the .po from main trunk to release this application,
that is not a big problem at the moment I think,
anyway, any questions from you is very helpful, thanks!
| Jin (jindallo) wrote : | # |
A newer commit is ongoing,
to merge trunk into the branch.
| Jin (jindallo) wrote : | # |
Hello Roberto,
I just submitted the r157: "Merge trunk" to this branch,
code rebased successfully, no conflicts now.
As I stated this change also includes the minor .po changes,
to modify the moderator from karni -> jin in the path,
since I will release main trunk with that after review pass.
Only reviewed code, not tested, but lgtm
| Jin (jindallo) wrote : | # |
Merge proposal had been patched according to Roberto's comment,
please have a look there and thanks for his feedback.
| David Planella (dpm) wrote : | # |
Please note that as translations are done in Launchpad and that no
translators modify the .po files directly, I think the best thing would be
to ignore the .po file changes. Especially as we've got LP set up *not* to
import .po file translations.
So translations go only in one direction Translator -> LP web UI -> bzr
Cheers,
David.
On Fri, Mar 11, 2016 at 9:59 AM, Jin <email address hidden> wrote:
> Merge proposal had been patched according to Roberto's comment,
> please have a look there and thanks for his feedback.
> --
>
> https:/
> Your team libqtelegram team is subscribed to branch lp:telegram-app.
>
| Jin (jindallo) wrote : | # |
Hello David,
Thanks for your reminder,
I just had a merge proposal for doing this according to your comment,
please help me to approve that if no any concerns/problems from you,
really thanks.
https:/


There are conflicts... Also I see there are 64755 modified lines, most of them are related with the .po files. Is that right?