Burgx swipe by Kirillvs · Pull Request #22 · LtdArink-Group/DeliveryApp

Kirillvs · 2018-08-07T20:19:14Z

No description provided.

This reverts commit 9043dd0.

Kirillvs · 2018-08-07T20:33:59Z

app/build.gradle

            resValue "string", "app_name", "Burgx Demo Local"
            // For android emulator local machine is 10.0.2.2
-            buildConfigField "String", "SERVER_URL", '"http://10.0.2.2:3000/"'
+            buildConfigField "String", "SERVER_URL", '"http://23.101.67.216:8080/"'


Зачем это меняешь?
10.0.2.2 это адрес, по которому доступна хост машина с эмулятора, то есть если ты развернешь у себя на компе сервак, сможешь его использовать
В Gradle уже все варианты указаны
Используются через вкладку Build Variants в Android Studio
Если хочешь, что бы приложение смотрело на:

staging выбирай demoStagingDebug

production выбирай demoProductionDebug

на твой комп выбирай demoLocalDebug

Когда будешь делать релизный билд, то выбирай demoProductionRelease

Это Андрей менял, потому что у меня в эмуляторе приложение не могло загрузить данные из базы

Кирилл, я не знаю что там надо делать чтобы эмулятор смотрел на демо. У Славы была винда, и бекенд они бы там хрен развернули ) Я пошел по простому пути.

Но вообще да, оставлять такие изменения врядли хорошая идея...

Kirillvs · 2018-08-07T20:41:01Z

app/src/main/java/ru/arink_group/deliveryapp/domain/dao/News.java

@@ -0,0 +1,33 @@
+package ru.arink_group.deliveryapp.domain.dao;
+
+public class News {


Имхо для такого data access object удобнее использовать data class из kotlin, он за тебя сделает все сеттеры, геттеры

Учту, спасибо =)

Kirillvs · 2018-08-07T20:43:08Z

app/src/main/java/ru/arink_group/deliveryapp/domain/dao/Order.kt

+        val deliveryTime: String,
+        val note: String?,
+        val pickup: Boolean,
+        val products: MutableList<OrderProduct>,


А зачем меняешь на MutableList?

Это я не менял насколько я помню

Гиту виднее кто что менял =)

Kirillvs · 2018-08-07T20:47:14Z

app/src/main/java/ru/arink_group/deliveryapp/domain/dao/Order.kt

        val dateTime = DateTime(date)
-
+        if (dateTime.minute == 59 && dateTime.hour == 22)
+            return true


Почему он должен быть активен в 22:59? 🤔

Все, что связано с возвратом из таймпикера в случае выбора ближайшего времени и местами, где есть ==22 и ==59, это сделано так, потому что в версии приложения, с которой я работал, бэк не был готов к передаче null, так Андрей сказал мне, поэтому я сделал костыль ,надо смерджить версии и тогда можно будет сделать null вместо этого костыля, ну а вообще мне конечно надо было хоть комментарий оставить на этот счет, извините

Вообще проблемы бекенда - это проблемы бекенда. Не нужно делать костыли "пока бекенд не заработает". Было ведь обговорено что должно быть так. Сейчас уже принимает

тогда я делаю возврат null, но тогда в моей версии приложения , до соединения с вашей, будут ошибки несоответствия типов, потому что сейчас мне пишет, что там должен быть Int, а я пытаюсь передать Null, соответственно работать оно не будет, пока не соедините с той версией, где уже можно Null передавать в DateTime

Kirillvs · 2018-08-07T20:49:37Z

app/src/main/java/ru/arink_group/deliveryapp/domain/dto/NewsDTO.java

+    @SerializedName("description")
+    @Expose
+    private String description;
+    @SerializedName("imgUrl")


А точно так? Мне кажется с бэка тебе придет img_url, а не imgUrl

это я поменяю

Kirillvs · 2018-08-07T21:58:22Z

app/src/main/java/ru/arink_group/deliveryapp/presentation/view/fragment/OrderFragment.java


+    @Override
+    public String getNote() {
+        return (note.getText().toString().equals("Комментарий ...")) ? null : note.getText().toString();


там же плейсхолдер какой то Комментарий ... ?
Лучше вынести и использовать из R.string или какого то общего места, что бы один источник был

Комментарий... это hint к edittext. Я делал это ссылкой на строку, но у меня не получилось почему-то добиться правильной работы, потому что я вытягивал из R.string.note постоянно id строки , а не саму строку

Kirillvs · 2018-08-07T22:06:48Z

app/src/main/java/ru/arink_group/deliveryapp/presentation/view/fragment/OrderFragment.java

    }
+
+    @Override
+    public void onTimeChanged(TimePicker view, int hourOfDay, int minute) {


я так понимаю теперь вместо onTimeSet используется onTimeChanged, тогда надо выпилить метод и TimePickerDialog.OnTimeSetListener из implements
или оно где то еще используется?

Kirillvs · 2018-08-07T22:08:14Z

app/src/main/java/ru/arink_group/deliveryapp/presentation/view/fragment/RetryOrderFragment.kt

 * A simple [Fragment] subclass.
 */
-class RetryOrderFragment : Fragment(), RetryOrderView, TimePickerDialog.OnTimeSetListener {
+class RetryOrderFragment : Fragment(), RetryOrderView, TimePickerDialog.OnTimeSetListener, TimePicker.OnTimeChangedListener {


тут аналогично, если интерфейс TimePickerDialog.OnTimeSetListener не используется то нужно выпилить и его и сам метод реализованный

да, уберу

Kirillvs · 2018-08-07T22:09:02Z

app/src/main/java/ru/arink_group/deliveryapp/presentation/view/fragment/RetryOrderFragment.kt

        val dateTime = DateTime(date)
-
-        timePicker.text = dateTime.toTimeWithDate()
+        if (dateTime.minute == 59 && dateTime.hour == 22)


Kirillvs · 2018-08-07T22:13:17Z

local.properties

+# For customization when using a Version Control System, please read the
+# header note.
+#Wed Jul 04 15:41:42 VLAT 2018
+sdk.dir=C\:\\Users\\Swipe\\AppData\\Local\\Android\\Sdk


Так как у каждого свои настройки среды разработки такие файлы нельзя добавлять, добавь в .gitignore и выпили из репозитория

я не понял, что надо сделать, что значит добавить в gitignore и выпилить из репозитория

В проекте есть файл .gitignore лежит в корне. В нем правила прописаны какие файлы не должны быть в гите.
Соответственно нужно добавить правило в этот файл. И затем выпилить его из гита

This reverts commit 9043dd0.

Swipesh added 3 commits August 7, 2018 15:57

Add files via upload

46d85a9

Add files via upload

9043dd0

Revert "Add files via upload"

522656a

This reverts commit 9043dd0.

Kirillvs commented Aug 7, 2018

View reviewed changes

Revert "Add files via upload"

d0555ec

This reverts commit 9043dd0.

		@@ -0,0 +1,33 @@
		package ru.arink_group.deliveryapp.domain.dao;

		public class News {

Conversation

Kirillvs commented Aug 7, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Swipesh Aug 8, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Swipesh Aug 8, 2018 •

edited

Loading