From 4b87e741238b34b93c110e1aa508b9b256c17f3d Mon Sep 17 00:00:00 2001 From: phansier Date: Fri, 23 Feb 2018 23:22:02 +0300 Subject: [PATCH 1/8] Revert: Remove unnecessary string (login_or_email) --- app/src/main/res/values-bg/strings.xml | 1 + app/src/main/res/values-el/strings.xml | 1 + app/src/main/res/values-es/strings.xml | 1 + app/src/main/res/values-fr/strings.xml | 1 + app/src/main/res/values-hu/strings.xml | 1 + app/src/main/res/values-it/strings.xml | 1 + app/src/main/res/values-pt/strings.xml | 1 + app/src/main/res/values-sv/strings.xml | 1 + app/src/main/res/values-uk/strings.xml | 1 + app/src/main/res/values-zh-rCN/strings.xml | 1 + app/src/main/res/values-zh-rTW/strings.xml | 1 + 11 files changed, 11 insertions(+) diff --git a/app/src/main/res/values-bg/strings.xml b/app/src/main/res/values-bg/strings.xml index 2e12efa1..1cd32577 100644 --- a/app/src/main/res/values-bg/strings.xml +++ b/app/src/main/res/values-bg/strings.xml @@ -170,6 +170,7 @@ Моля, въведете коректно име и парола Моля, въведете коректна парола. Парола + Име или имейл Фенове Следва Фенове diff --git a/app/src/main/res/values-el/strings.xml b/app/src/main/res/values-el/strings.xml index 34e96905..8d167b01 100644 --- a/app/src/main/res/values-el/strings.xml +++ b/app/src/main/res/values-el/strings.xml @@ -186,6 +186,7 @@ Παρακαλούμε εισάγετε έγκυρο όνομα χρήστη & συνθηματικό Παρακαλούμε εισάγετε έγκυρο συνθηματικό. Συνθηματικό + Όνομα ή Email Ακόλουθοι Ακολουθούν Ακόλουθοι diff --git a/app/src/main/res/values-es/strings.xml b/app/src/main/res/values-es/strings.xml index d6f29962..b70ab207 100644 --- a/app/src/main/res/values-es/strings.xml +++ b/app/src/main/res/values-es/strings.xml @@ -186,6 +186,7 @@ Por favor introduce un usuario y contraseña válidos Por favor introduce una contraseña válida. Contraseña + Usuario o Correo Electrónico Seguidores Siguiendo Seguidores diff --git a/app/src/main/res/values-fr/strings.xml b/app/src/main/res/values-fr/strings.xml index b1162a80..5e88f500 100644 --- a/app/src/main/res/values-fr/strings.xml +++ b/app/src/main/res/values-fr/strings.xml @@ -178,6 +178,7 @@ Veuillez entrer un login et un mot de passe valides Veuillez entrer un mot de passe valide Mot de passe + Login ou Email Abonnés Abonnements Abonnés diff --git a/app/src/main/res/values-hu/strings.xml b/app/src/main/res/values-hu/strings.xml index b8f2db2d..56f26be5 100644 --- a/app/src/main/res/values-hu/strings.xml +++ b/app/src/main/res/values-hu/strings.xml @@ -166,6 +166,7 @@ Adj meg érvényes felhasználónevet & jelszót Adj meg érvényes jelszót. Jelszó + Felhasználónév vagy E-mail cím Követők Követettek Követőim diff --git a/app/src/main/res/values-it/strings.xml b/app/src/main/res/values-it/strings.xml index ea637657..ad46a223 100644 --- a/app/src/main/res/values-it/strings.xml +++ b/app/src/main/res/values-it/strings.xml @@ -179,6 +179,7 @@ Inserisci un nome utente e una password validi Inserisci una password valida. Password + Login o Email Followers Following Followers diff --git a/app/src/main/res/values-pt/strings.xml b/app/src/main/res/values-pt/strings.xml index 83bf34e0..280c9a6f 100644 --- a/app/src/main/res/values-pt/strings.xml +++ b/app/src/main/res/values-pt/strings.xml @@ -162,6 +162,7 @@ Por favor, digite um login e senha válidos. Por favor, digite uma senha válida. Senha + Login ou Email Seguidores Seguindo Membros diff --git a/app/src/main/res/values-sv/strings.xml b/app/src/main/res/values-sv/strings.xml index b131c996..ce59d4a5 100644 --- a/app/src/main/res/values-sv/strings.xml +++ b/app/src/main/res/values-sv/strings.xml @@ -192,6 +192,7 @@ Ange ett giltligt lösenord. Lösenord Användarnamn eller Email + Användarnamn eller Email Följare följar på följa diff --git a/app/src/main/res/values-uk/strings.xml b/app/src/main/res/values-uk/strings.xml index c9acb0cd..d252400a 100644 --- a/app/src/main/res/values-uk/strings.xml +++ b/app/src/main/res/values-uk/strings.xml @@ -175,6 +175,7 @@ Введіть правильні логін та пароль Введіть правильний пароль. Пароль + Логін або пошта Читачі Підписки Читачі diff --git a/app/src/main/res/values-zh-rCN/strings.xml b/app/src/main/res/values-zh-rCN/strings.xml index 90a014a3..e545873c 100644 --- a/app/src/main/res/values-zh-rCN/strings.xml +++ b/app/src/main/res/values-zh-rCN/strings.xml @@ -237,6 +237,7 @@ 请输入正确的账户名和密码。 请输入正确的密码。 密码 + 账户名 / 电子邮件 关注者 正在关注 关注我的 diff --git a/app/src/main/res/values-zh-rTW/strings.xml b/app/src/main/res/values-zh-rTW/strings.xml index 844ed3d3..4e8eb4ce 100644 --- a/app/src/main/res/values-zh-rTW/strings.xml +++ b/app/src/main/res/values-zh-rTW/strings.xml @@ -162,6 +162,7 @@ 請輸入正確的帳號與密碼 請輸入正確的密碼. 密碼 + 帳號或電子郵件 追隨我的 我追隨的 追隨我的 From e3a85c23a2c4a31911bb2019645bd8c6019eb698 Mon Sep 17 00:00:00 2001 From: phansier Date: Fri, 23 Feb 2018 23:27:09 +0300 Subject: [PATCH 2/8] Revert: Change milestone translation in RU on more appropriate --- app/src/main/res/values-ru/strings.xml | 30 +++++++++++++------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/app/src/main/res/values-ru/strings.xml b/app/src/main/res/values-ru/strings.xml index 852a8154..522fd8e2 100644 --- a/app/src/main/res/values-ru/strings.xml +++ b/app/src/main/res/values-ru/strings.xml @@ -34,7 +34,7 @@ Ошибка при загрузке Gists Ошибка при загрузке задачи Ошибка при загрузке соучастников - Ошибка при загрузке вех + Ошибка при загрузке целей Ошибка при загрузке тегов Ошибка при загрузке закладок Ошибка при загрузке членов @@ -73,7 +73,7 @@ Загружаем Gists… Загружаем соучастников… Загружаем сотрудников… - Загружаем вехи… + Загружаем цели… Загружаем теги… Загружаем коммиты… Загружаем файлы и комментарии… @@ -104,7 +104,7 @@ Обновляем ответственного… Обновляем задачу… Обновляем теги… - Обновляем веху… + Обновляем цель… @@ -114,8 +114,8 @@ назначение себе снято добавлен %1$s тег удалён %1$s тег - добавлено к %1$s вехе - удалено из %1$s вехи + добавлено к %1$s цели + удалено из %1$s цели объединён коммит %1$s ссылается на задачу для коммита %1$s ссылается на задачу в %1$s @@ -183,8 +183,8 @@ Участники Задачи Редактировать теги - Веха: - Изменить веху + Цель: + Изменить цель Назначить ответственного Описание Gist, созданный на Android @@ -194,7 +194,7 @@ Снимаем отметку с Gist… Аккаунты Выбрать ответственного - Выбрать веху + Выбрать цель Выбрать тег Выбрать ветку или метку ForkHub будет использовать ваш пароль только для генерации <i>Personal Access Token</i>. Вы можете <a href=\"https://github.com/settings/tokens\">создать его самостоятельно</a> и использовать его здесь вместо пароля. @@ -202,8 +202,8 @@ Для вашего аккаунта включена двухэтапная аутентификация. Введите код аутентификации для подтверждения личности. Если вам нравится это приложение, пожалуйста оцените его звёздами. Если вам нравится это приложение, пожалуйста оцените его рейтингом Play Store. - Оценить - Нет вех + Rate + Нет целей Нет ответственных ответственен Gists не найдены @@ -229,7 +229,7 @@ Закрыта Ответственный: Кто угодно - Веха: + Цель: Нет Теги: Войти @@ -272,7 +272,7 @@ ОК Теги: Ответственный - Вехи + Цель Теги Вы действительно хотите закрыть эту задачу? Вы действительно хотите переоткрыть эту задачу? @@ -358,11 +358,11 @@ Просрочен дней Выбрать задачу - Новая веха + Новая цель Ошибка при сохранении с пустым заголовком - Создаем веху… - Удаляем веху… + Создаем цель… + Удаляем цель… начата проверка Родительский репозиторий From 9e5ae56fd8dd1d9b8940cf02bbb156f60531f373 Mon Sep 17 00:00:00 2001 From: phansier Date: Fri, 23 Feb 2018 23:35:41 +0300 Subject: [PATCH 3/8] Use new Milestone model in IssueFilterTest --- .../com/github/mobile/tests/issue/IssueFilterTest.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/app/src/androidTest/java/com/github/mobile/tests/issue/IssueFilterTest.java b/app/src/androidTest/java/com/github/mobile/tests/issue/IssueFilterTest.java index f129f194..bd3767ba 100644 --- a/app/src/androidTest/java/com/github/mobile/tests/issue/IssueFilterTest.java +++ b/app/src/androidTest/java/com/github/mobile/tests/issue/IssueFilterTest.java @@ -17,9 +17,9 @@ import android.test.AndroidTestCase; +import com.github.mobile.api.model.Milestone; import com.github.mobile.core.issue.IssueFilter; -import org.eclipse.egit.github.core.Milestone; import org.eclipse.egit.github.core.Repository; import org.eclipse.egit.github.core.User; @@ -56,11 +56,11 @@ public void testEqualFilter() { assertEquals(filter1, filter2); assertEquals(filter1.hashCode(), filter2.hashCode()); - Milestone milestone = new Milestone().setNumber(3); - com.github.mobile.api.model.Milestone extraMilestone = new com.github.mobile.api.model.Milestone(milestone); - filter1.setMilestone(extraMilestone); + Milestone milestone = new Milestone(); + milestone.number = 3; + filter1.setMilestone(milestone); assertFalse(filter1.equals(filter2)); - filter2.setMilestone(extraMilestone); + filter2.setMilestone(milestone); assertEquals(filter1, filter2); assertEquals(filter1.hashCode(), filter2.hashCode()); } From 30048f0c7de023a3dcc3423047d0d5c696b5be89 Mon Sep 17 00:00:00 2001 From: phansier Date: Fri, 23 Feb 2018 23:37:06 +0300 Subject: [PATCH 4/8] Fix Whitespace missing before { --- app/src/main/java/com/github/mobile/api/model/Issue.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/com/github/mobile/api/model/Issue.java b/app/src/main/java/com/github/mobile/api/model/Issue.java index 368c495b..50337028 100644 --- a/app/src/main/java/com/github/mobile/api/model/Issue.java +++ b/app/src/main/java/com/github/mobile/api/model/Issue.java @@ -77,6 +77,7 @@ public org.eclipse.egit.github.core.Issue getOldModel() { issue.setCreatedAt(created_at); issue.setClosedAt(closed_at); issue.setUpdatedAt(updated_at); + if (milestone != null){ issue.setMilestone(milestone.getOldModel()); } @@ -92,4 +93,4 @@ public org.eclipse.egit.github.core.Issue getOldModel() { } return issue; } -} +} \ No newline at end of file From 0883b55c7981aa72a4a8ef8a057ed705fa7db707 Mon Sep 17 00:00:00 2001 From: phansier Date: Fri, 23 Feb 2018 23:38:29 +0300 Subject: [PATCH 5/8] Remove comments about Milestone new to old model mapping issues --- app/src/main/java/com/github/mobile/api/model/Milestone.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/app/src/main/java/com/github/mobile/api/model/Milestone.java b/app/src/main/java/com/github/mobile/api/model/Milestone.java index d4b436a2..dcc6231a 100644 --- a/app/src/main/java/com/github/mobile/api/model/Milestone.java +++ b/app/src/main/java/com/github/mobile/api/model/Milestone.java @@ -51,10 +51,6 @@ public Milestone() { } public Milestone(org.eclipse.egit.github.core.Milestone milestone) { - //todo this.id=? - //there are two variants of id usage - //1)id is like an id in Issues, but it is in old Issues class too - the remove field - //2)id is like an serialVersionUID - then generate it using "serialver -classpath . Milestone" in terminal this.number = milestone.getNumber(); this.state = milestone.getState(); this.title = milestone.getTitle(); @@ -65,7 +61,6 @@ public Milestone(org.eclipse.egit.github.core.Milestone milestone) { this.closed_issues = milestone.getClosedIssues(); this.created_at = milestone.getCreatedAt(); this.url = milestone.getUrl(); - //todo this.updated_at=??? this.due_on = milestone.getDueOn(); } From f493fe989ba432b792b9f99db2de85005fa00bb8 Mon Sep 17 00:00:00 2001 From: phansier Date: Fri, 23 Feb 2018 23:53:36 +0300 Subject: [PATCH 6/8] Make use the plural "Milestones" here (ru+en locals) --- app/src/main/res/menu/repository.xml | 2 +- app/src/main/res/values-ru/strings.xml | 1 + app/src/main/res/values/strings.xml | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) diff --git a/app/src/main/res/menu/repository.xml b/app/src/main/res/menu/repository.xml index 6877c950..a8103379 100644 --- a/app/src/main/res/menu/repository.xml +++ b/app/src/main/res/menu/repository.xml @@ -33,7 +33,7 @@ + android:title="@string/milestones"/> Выбрать задачу Новая цель Ошибка при сохранении с пустым заголовком + Цели Создаем цель… Удаляем цель… diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 8c4bd03a..6b36110a 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -365,6 +365,7 @@ dd MMM yyyy Select Issue Saving with empty title failed + Milestones From 19d4db3fa76b97149d2fe38d46f78994fe081887 Mon Sep 17 00:00:00 2001 From: phansier Date: Sat, 24 Feb 2018 00:18:16 +0300 Subject: [PATCH 7/8] Avoid NPE in getting old milestone --- .../github/mobile/ui/issue/EditIssuesFilterActivity.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/com/github/mobile/ui/issue/EditIssuesFilterActivity.java b/app/src/main/java/com/github/mobile/ui/issue/EditIssuesFilterActivity.java index c6c491eb..c2e60b2b 100644 --- a/app/src/main/java/com/github/mobile/ui/issue/EditIssuesFilterActivity.java +++ b/app/src/main/java/com/github/mobile/ui/issue/EditIssuesFilterActivity.java @@ -145,7 +145,12 @@ public void onClick(View v) { milestoneDialog = new MilestoneDialog( EditIssuesFilterActivity.this, REQUEST_MILESTONE, repository, milestones); - milestoneDialog.show(filter.getMilestone().getOldModel()); + com.github.mobile.api.model.Milestone milestone = filter.getMilestone(); + if (milestone == null) { + milestoneDialog.show(null); + } else { + milestoneDialog.show(milestone.getOldModel()); + } } }; From f2044a24ac54e67d25af252df009f598329f4a89 Mon Sep 17 00:00:00 2001 From: Nikita Tsyrlin Date: Sat, 24 Feb 2018 19:05:38 +0300 Subject: [PATCH 8/8] Remove redundant headers --- .../com/github/mobile/api/service/MilestoneService.java | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/app/src/main/java/com/github/mobile/api/service/MilestoneService.java b/app/src/main/java/com/github/mobile/api/service/MilestoneService.java index 58d02de5..dbff0cae 100644 --- a/app/src/main/java/com/github/mobile/api/service/MilestoneService.java +++ b/app/src/main/java/com/github/mobile/api/service/MilestoneService.java @@ -30,14 +30,12 @@ import retrofit2.http.Query; public interface MilestoneService { - @Headers("Accept: application/vnd.github.v3+json") @GET("repos/{owner}/{repo}/milestones/{number}") Call getMilestone( @Path("owner") String owner, @Path("repo") String repo, @Path("number") long number); - @Headers("Accept: application/vnd.github.v3+json") @GET("repos/{owner}/{repo}/milestones") Call> getMilestones( @Path("owner") String owner, @@ -50,14 +48,12 @@ Call> getIssues( @Path("repo") String repo, @Query("milestone") long milestone); - @Headers("Accept: application/vnd.github.v3+json") @POST("repos/{owner}/{repo}/milestones") Call createMilestone ( @Path("owner") String owner, @Path("repo") String repo, @Body Milestone milestone); - - @Headers("Accept: application/vnd.github.v3+json") + @PATCH("repos/{owner}/{repo}/milestones/{number}") Call editMilestone ( @Path("owner") String owner, @@ -65,7 +61,6 @@ Call editMilestone ( @Path("number") long number, @Body Milestone milestone); - @Headers("Accept: application/vnd.github.v3+json") @DELETE("repos/{owner}/{repo}/milestones/{number}") Call deleteMilestone( @Path("owner") String owner,