Country prediction during series creation #547
Conversation
Generated by 🚫 Danger |
|
а как такие ошибки исправлять линк <http://prntscr.com/ejzlfx> ?
|
|
Не очень понятно, что это такое так как под Linux/MacOS этих ошибок нет. Либо это баг в самом checkstyle, либо его надо как-то специальным образом конфигурировать под Windows. Поэтому можешь игнорировать. Как вариант -- заведи задачу, чтобы при возможности я посмотрел или завел баг в апстриме. |
There was a problem hiding this comment.
Посмотрел более-менее внимательно и оставил комментарии. Кажется, не смог удержаться и все-таки обратил твое внимание на мелкие детали, но они должны быть очень легко устраняемы.
Кстати, у тебя почему-то везде фигурирует задача 571, но такой еще нет, есть 517 :) Если вдруг соберешься ветку переименовывать (не советую, но вдруг ты захочешь), то не пересоздавай запрос, а просто измени в нем ветку.
| @@ -11,5 +11,6 @@ | |||
| <include file="initial-state.xml" relativeToChangelogFile="true" /> | |||
| <include file="version/0.3.xml" relativeToChangelogFile="true" /> | |||
| <include file="version/0.4.xml" relativeToChangelogFile="true" /> | |||
| <include file="initial-test.xml" relativeToChangelogFile="true" /> | |||
There was a problem hiding this comment.
Давай уберем миграцию и тестовые данные для начала, чтобы сфокусироваться на коде. А тесты добавим чуть позже, отдельно.
| @@ -121,6 +121,7 @@ t_add_more_images_hint = Later you will be able to add additional images | |||
| t_not_chosen = Not chosen | |||
| t_create_category_hint = You can also <a tabindex="-1" href="{0}">add a new category</a> | |||
| t_create_country_hint = You can also <a tabindex="-1" href="{0}">add a new country</a> | |||
| t_guess_country = Guess the country | |||
| @@ -202,6 +202,10 @@ <h3 th:text="${#strings.capitalize(add_series)}"> | |||
| <span id="country.errors" class="help-block" th:if="${#fields.hasErrors('country')}" th:each="error : ${#fields.errors('country')}" th:text="${error}"></span> | |||
| /*/--> | |||
| </div> | |||
|
|
|||
| <span id="guess_country"> | |||
| <a tabindex="-1" th:text="#{t_guess_country}" href="javascript:void(0)">Guess the country</a> | |||
| @Override | ||
| public String guessCountryBy(Integer createdBy) { | ||
| Map<String, Object> params = new HashMap<>(); | ||
| params.put("created_by", createdBy); |
There was a problem hiding this comment.
Используй Collections.singletonMap() раз у тебя всего одно значение.
| @@ -421,6 +422,13 @@ public String searchSeriesByCatalog( | |||
| return "series/search_result"; | |||
| } | |||
|
|
|||
| @ResponseBody | |||
There was a problem hiding this comment.
Добавь комментарий со своим авторством.
| @@ -202,6 +202,10 @@ <h3 th:text="${#strings.capitalize(add_series)}"> | |||
| <span id="country.errors" class="help-block" th:if="${#fields.hasErrors('country')}" th:each="error : ${#fields.errors('country')}" th:text="${error}"></span> | |||
| /*/--> | |||
| </div> | |||
|
|
|||
| <span id="guess_country"> | |||
There was a problem hiding this comment.
Нужно добавить новый переключатель в togglz и скрывать и этот тег и JS-код для него, в случае, если фича выключена.
| @@ -421,6 +422,13 @@ public String searchSeriesByCatalog( | |||
| return "series/search_result"; | |||
| } | |||
|
|
|||
| @ResponseBody | |||
| @GetMapping(Url.INFO_COUNTRY_SERIES_PAGE) | |||
There was a problem hiding this comment.
Вынеси метод в отдельный контроллер (SuggestionController).
| @@ -223,6 +223,14 @@ public long countUpdatedSince(Date date) { | |||
|
|
|||
| @Override | |||
| @Transactional(readOnly = true) | |||
| public String guessCountryBy(Integer createdBy) { | |||
There was a problem hiding this comment.
Нужно добавить защиту на метод. Он должен быть доступен только пользователям с авторити CREATE_SERIES.
| SELECT slug \ | ||
| FROM series LEFT JOIN countries ON countries.id = series.country_id \ | ||
| GROUP BY country_id \ | ||
| ORDER BY COUNT(*) DESC LIMIT 1 |
There was a problem hiding this comment.
Кажется, что этот запрос выбирает что-то отличное от того, чтобы было в ТЗ. Наша задача найти самую популярную страну в коллекции пользователя.
| FROM series LEFT JOIN countries ON countries.id = series.country_id \ | ||
| WHERE series.created_at IN (SELECT MAX(series.created_at) \ | ||
| FROM series \ | ||
| WHERE series.created_by = :created_by) |
There was a problem hiding this comment.
Надо будет еще раз потом (не в два ночи) посмотреть на этот запрос. Какой-то он чересчур сложный, как мне кажется.
|
По поводу запросов:
1) Я сначала ищу последнию добавленную пользователем страну
SELECT slug \
FROM series LEFT JOIN countries ON countries.id = series.country_id \
WHERE series.created_at IN (SELECT MAX(series.created_at) \
FROM series \
WHERE series.created_by = :created_by)
Как вариант можно сделать вот так:
SELECT slug
FROM series LEFT JOIN countries ON countries.id = series.country_id
WHERE series.created_by = 1
ORDER BY series.created_at DESC LIMIT 1
2) Если пользователей не добавлял страну, тогда ищу самую популярную страну
в таблице.
… +series.find_popular_country = \
+ SELECT slug \
+ FROM series LEFT JOIN countries ON countries.id = series.country_id \
+GROUP BY country_id \
+ORDER BY COUNT(*) DESC LIMIT 1
|
php-coder
changed the title
php-coder
changed the title
|
Привет, можешь подсказать ссылка <http://prntscr.com/elfte8> (цифра 1) я
пытаюсь вот так передать параметр, но он не подставляет строку(( но под
цифрой 2 он подставляет все корректно. Что не так?
|
Shkarin
force-pushed
the
gh571_prediction_country
branch
from
9785734 to
5350d35
Compare
| @@ -15,7 +15,7 @@ | |||
| <link rel="stylesheet" href="../../static/styles/main.css" th:href="${MAIN_CSS}" /> | |||
| <link rel="stylesheet" href="http://cdnjs.cloudflare.com/ajax/libs/selectize.js/0.12.3/css/selectize.bootstrap3.min.css" th:href="${SELECTIZE_CSS}" /> | |||
| </head> | |||
| <body onload="initPage()"> | |||
| <body> | |||
There was a problem hiding this comment.
Hm... did you try <body th:onload="initPage(${urlSuggest})"> ?
|
Забыл добавить, что нужно добавить фичу в togglz, чтобы эту ссылку и js-код можно было отключать динамически. |
Чтобы подавить подобные предупреждения добавь |
Shkarin
force-pushed
the
gh571_prediction_country
branch
3 times, most recently
from
f1b685a to
5254bb1
Compare
| */ | ||
| @ResponseBody | ||
| @GetMapping(Url.INFO_COUNTRY_SERIES_PAGE) | ||
| @PreAuthorize(HasAuthority.CREATE_SERIES) |
There was a problem hiding this comment.
Эту аннотацию нужно вешать на сервис (на случай, если в другой раз мы вызовим его из другого контроллера). А контроллер нужно защищать немного иначе -- см. src/main/java/ru/mystamps/web/support/spring/security/SecurityConfig.java
| @Override | ||
| @Transactional(readOnly = true) | ||
| public String suggestCountryForUser(Integer userId) { | ||
| Validate.isTrue(userId != null, "UserId must be non null"); |
There was a problem hiding this comment.
Поправь чуток описание: "User id must be non null"
| @@ -0,0 +1,12 @@ | |||
| -- | |||
| @@ -0,0 +1,18 @@ | |||
| -- | |||
| @@ -15,7 +15,7 @@ | |||
| <link rel="stylesheet" href="../../static/styles/main.css" th:href="${MAIN_CSS}" /> | |||
| <link rel="stylesheet" href="http://cdnjs.cloudflare.com/ajax/libs/selectize.js/0.12.3/css/selectize.bootstrap3.min.css" th:href="${SELECTIZE_CSS}" /> | |||
| </head> | |||
| <body onload="initPage()"> | |||
| <body onload="initPage(${suggestCountryUrl})"> | |||
There was a problem hiding this comment.
Думаю, что вот эта строчка -- причина падения тестов. После отрисовки на сервере, на клиент придет код типа такого: <body onload="initPage(/suggestion/country)"> Этот код, разумеется, невалидный, так как строки должны быть в кавычках.
| @@ -81,3 +81,17 @@ country.find_country_link_info_by_slug = \ | |||
| FROM countries c \ | |||
| WHERE c.slug = :slug \ | |||
| ORDER BY CASE WHEN 'ru' = :lang THEN COALESCE(c.name_ru, c.name) ELSE c.name END | |||
|
|
|||
| country.find_last_country_by_id = \ | |||
| SELECT slug \ | |||
There was a problem hiding this comment.
При использовании алиасов лучше везде потом явно указывать из какой таблицы поля, чтобы а) было сразу понятно при чтении запроса б) этот SQL не стал невалидным, если во второй таблице появится такое же поле.
| @@ -31,6 +31,7 @@ | |||
| @SuppressWarnings("PMD.TooManyMethods") | |||
| public interface SeriesService { | |||
| Integer add(AddSeriesDto dto, Integer userId, boolean userCanAddComments); | |||
|
|
|||
There was a problem hiding this comment.
Не забудь убрать эту пустую строку.
| @@ -30,6 +30,7 @@ | |||
| @SuppressWarnings("PMD.TooManyMethods") | |||
| public interface SeriesDao { | |||
| Integer add(AddSeriesDbDto series); | |||
|
|
|||
There was a problem hiding this comment.
Не забудь убрать пустую строку тут.
|
In src/main/java/ru/mystamps/web/controller/SuggestionController.java
<#547 (comment)>:
+import ru.mystamps.web.controller.converter.annotation.CurrentUser;
+import ru.mystamps.web.service.CountryService;
+import ru.mystamps.web.support.spring.security.HasAuthority;
+
+@controller
+@requiredargsconstructor
+@SuppressWarnings({ "PMD.AvoidDuplicateLiterals",
"PMD.TooManyMethods", "PMD.GodClass" })
+public class SuggestionController {
+
+ private final CountryService countryService;
+ /**
+ * @author John Shkarin
+ */
+ @responsebody
+ @GetMapping(Url.INFO_COUNTRY_SERIES_PAGE)
+ @PreAuthorize(HasAuthority.CREATE_SERIES)
Эту аннотацию нужно вешать на сервис (на случай, если в другой раз мы
вызовим его из другого контроллера). А контроллер нужно защищать немного
иначе -- см. src/main/java/ru/mystamps/web/support/spring/security/Se
curityConfig.java
Вот с этим не сообразил почему нужно брать из SecurityConfig? я делаю как в
CountryController
2017-03-27 2:34 GMT+05:00 Vyacheslav Semushin <[email protected]>:
… ***@***.**** commented on this pull request.
------------------------------
In src/main/java/ru/mystamps/web/dao/SeriesDao.java
<#547 (comment)>:
> @@ -30,6 +30,7 @@
@SuppressWarnings("PMD.TooManyMethods")
public interface SeriesDao {
Integer add(AddSeriesDbDto series);
+
Не забудь убрать пустую строку тут.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDATaW81o-m7sRHTJ1D7WRegsh7U7pks5rptoCgaJpZM4MdDEJ>
.
|
|
и вот еще <body onload="initPage("${suggestCountryUrl})"> как правильно
кавычки поставить чтоб заработало?
2017-03-27 23:56 GMT+05:00 Евгений Шкарин <[email protected]>:
… In src/main/java/ru/mystamps/web/controller/SuggestionController.java
<#547 (comment)>:
> +import ru.mystamps.web.controller.converter.annotation.CurrentUser;
+import ru.mystamps.web.service.CountryService;
+import ru.mystamps.web.support.spring.security.HasAuthority;
+
***@***.***
***@***.***
***@***.***({ "PMD.AvoidDuplicateLiterals", "PMD.TooManyMethods", "PMD.GodClass" })
+public class SuggestionController {
+
+ private final CountryService countryService;
+ /**
+ * @author John Shkarin
+ */
+ @responsebody
+ @GetMapping(Url.INFO_COUNTRY_SERIES_PAGE)
+ @PreAuthorize(HasAuthority.CREATE_SERIES)
Эту аннотацию нужно вешать на сервис (на случай, если в другой раз мы
вызовим его из другого контроллера). А контроллер нужно защищать немного
иначе -- см. src/main/java/ru/mystamps/web/support/spring/security/Se
curityConfig.java
Вот с этим не сообразил почему нужно брать из SecurityConfig? я делаю как
в CountryController
2017-03-27 2:34 GMT+05:00 Vyacheslav Semushin ***@***.***>:
> ***@***.**** commented on this pull request.
> ------------------------------
>
> In src/main/java/ru/mystamps/web/dao/SeriesDao.java
> <#547 (comment)>:
>
> > @@ -30,6 +30,7 @@
> @SuppressWarnings("PMD.TooManyMethods")
> public interface SeriesDao {
> Integer add(AddSeriesDbDto series);
> +
>
> Не забудь убрать пустую строку тут.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <#547 (review)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AVJDATaW81o-m7sRHTJ1D7WRegsh7U7pks5rptoCgaJpZM4MdDEJ>
> .
>
|
Попробуй так: |
Все правильно сделал. Просто есть две проверки -- 1) по URL-у, которая выполняется в самом начале и тогда выполнение не дойдет даже до контроллера 2) перед вызовом сервисного метода Нам нужны обе, ты пока реализовал вторую и я предлагаю добавить еще первую (в SecurityConfig). |
|
Так я пробовал, но не сработало( разным*и способами пробовал*
2017-03-28 1:08 GMT+05:00 Vyacheslav Semushin <[email protected]>:
… и вот еще <body onload="initPage("${suggestCountryUrl})"> как правильно
кавычки поставить чтоб заработало?
Попробуй так: <body onload="initPage('${suggestCountryUrl}')">
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDATDUaPiDSu4cA3JzXUMWr5nf1YG1ks5rqBdSgaJpZM4MdDEJ>
.
|
But then why I can see this link at all? It shouldn't be visible in this case. I bet that it doesn't work and request doesn't return |
|
Да есть, ошибка, $.get возвращает пустую строку, исправлю и добавлю скрытие
ссылки.
Только я что-то не так сделал и у меня появились в моей ветку кучу твоих
коммитов, это нормально или нужно октатить?
2017-04-17 22:10 GMT+05:00 Vyacheslav Semushin <[email protected]>:
… Потому что нет тестовых данных, запросы возвращают нул и ссылка просто
пропадает.
But then why I can see this link at all? It shouldn't be visible in this
case. I bet that it doesn't work and request doesn't return null as you
said. Try to inspect request in the browser inspector, for example. I know
why it doesn't work but I'll let you debug it first :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDAUavG1QbGv1ME4W825C3TGThHoVcks5rw50EgaJpZM4MdDEJ>
.
|
Будет любопытно взглянуть. Зачем добавлять скрытие, если она скрыта по-умолчанию? Насколько понимаю, нужно разобраться почему она раскрывается, когда с сервера не пришла нужная информация.
Я не знаю что ты делал и не знаю в каком состоянии сейчас ветка, так как ты это все не пушил на гитхаб, поэтому ничего не могу тебе подсказать. Скорее всего, да, ты сделал что-то странное. |
|
P.S. Ах, тот коммит преследует нас -- из-за того, что он все еще в твоем мастере, все работает так странно. Необходимо в ветке мастер сделать что-то типа |
|
P.P.S. Это прекрасный урок для меня больше никогда не менять публичные коммиты. Приношу свои извинения еще раз. |
|
Что-то у меня не получается: http://prntscr.com/exkakl
17 апреля 2017 г., 22:51 пользователь Vyacheslav Semushin <
[email protected]> написал:
… P.P.S. Это прекрасный урок для меня больше никогда не менять публичные
коммиты. Приношу свои извинения еще раз.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDAbLejqCtn8L9AG1Fs9DqO7Gw85rHks5rw6aagaJpZM4MdDEJ>
.
|
|
Во-первых, зачем второй раз делал git reset? Это конечно ничего не изменило, но выглядит странно. Во-вторых, похоже, что мой репо у тебя назван как-то иначе. Должен называться upstream, если ты следовал инструкции. Если ты назвал его как-то еще, то нужно подставить другое имя (см. |
|
Please, create a new PR from that branch, so we'll be able to continue working on this feature. |
|
А удалять коммит твой уже не стоит?
2017-04-19 22:39 GMT+05:00 Vyacheslav Semushin <[email protected]>:
… Please, create a new PR from that branch, so we'll be able to continue
working on this feature.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDAX4ExGmNElU91P6OaQlONHSNBKetks5rxkbigaJpZM4MdDEJ>
.
|
Стоит конечно. |
|
Если честно я смутно понимаю что мне нужно сделать и какой результат
получить. Во что я сделал http://prntscr.com/eyfxmt
2017-04-19 23:10 GMT+05:00 Vyacheslav Semushin <[email protected]>:
… А удалять коммит твой уже не стоит?
Стоит конечно.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDAW9iJYnG7b8WYtvRdWd9MEFQwo-Fks5rxk4tgaJpZM4MdDEJ>
.
|
|
Я пока не вижу результат полностью, но выглядит так, что ты делаешь все правильно. Но! Я сейчас понял, что у тебя, вероятно, этот коммит есть в Сделай тогда |
|
Давай я попробую объяснить. Есть следующие ветки
Теперь что получилось:
Для того, чтобы его удалить мы должны:
|
|
Спасибо. Я сделал, но результат такой же http://prntscr.com/eyg59q
20 апреля 2017 г., 1:39 пользователь Vyacheslav Semushin <
[email protected]> написал:
… Давай я попробую объяснить. Есть следующие ветки
1. master в моем репозитории на гитхабе
2. php-coder/master -- это отражение моего мастера из пункта 1 но
находящееся у тебя локально
3. master в твоем локальном репозитории -- это тоже самое что и мой
мастер, но основан он на локальной ветке из пункта 2
4. далее есть ветка gh571_prediction_country -- она основана на твоем
мастере плюс там твои изменения
Теперь что получилось:
- я добавил плохой коммит в 1)
- ты его скачал себе в 2)
- потом при мердже из 2) он также попал и в 3)
- далее при ребейзе он также попал и в 4)
Для того, чтобы его удалить мы должны:
- убрать его из 2) -- это должен сделать git fetch
- убрать его из 3) -- это должны сделать git reset и git merge
- убрать его из 4) -- это должен будет сделать git rebase
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDAeuuo9AxBcrQ9da1PcbgT3J_kNjCks5rxnEYgaJpZM4MdDEJ>
.
|
|
Очень странно. А что, например, выводит git log php-coder/master? |
|
1) Вот что выдает http://prntscr.com/eygatp
2) Пока не забыл, когда отключаешь js в браузере то кнопки становятся не
кликабельными http://prntscr.com/eygb5g
2017-04-20 1:53 GMT+05:00 Vyacheslav Semushin <[email protected]>:
… Очень странно. А что, например, выводит git log php-coder/master?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDATLE2X_K5napEdm08Ha27UB364kqks5rxnQ5gaJpZM4MdDEJ>
.
|
|
Попробуй сделать git fetch --force php-coder -- нужно чтобы твой php-coder/master бранч перезатерся изменениями из моего мастера. Еще меня смущает это сообщение про ambiguous refname... впервые такое вижу. |
Это баг. Можешь создать его? |
|
Да, попробую создать задачу и исправлю ошибку
20 апреля 2017 г., 2:07 пользователь Vyacheslav Semushin <
[email protected]> написал:
…
1. Пока не забыл, когда отключаешь js в браузере то кнопки становятся
не
кликабельными http://prntscr.com/eygb5g
Это баг. Можешь создать его?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDARXJ_NIn5xnYfaUqk_6b6YzeTt4aks5rxneLgaJpZM4MdDEJ>
.
|
|
Тоже самое:( http://prntscr.com/eygjjs
20 апреля 2017 г., 2:04 пользователь Vyacheslav Semushin <
[email protected]> написал:
… Попробуй сделать git fetch --force php-coder -- нужно чтобы твой
php-coder/master бранч перезатерся изменениями из моего мастера.
Еще меня смущает это сообщение про ambiguous refname... впервые такое вижу.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDASmv3Utt1cwHSpuUIJz3EVJViNKvks5rxnbugaJpZM4MdDEJ>
.
|
|
Окей, покажи, что выводят команды git tag и git branch -a |
|
Вот http://prntscr.com/eygn4w
2017-04-20 2:21 GMT+05:00 Vyacheslav Semushin <[email protected]>:
… Окей, покажи, что выводят команды git tag и git branch -a
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDAX2SVK9vEahByLvEwqNKD3faTEotks5rxnrRgaJpZM4MdDEJ>
.
|
|
Теперь мы нашли причину ambiguous refname -- у тебя есть локальная ветка php-coder/master которая затеняет такую же но remote (вот он: http://stackoverflow.com/a/28100234/352708). Сделай git branch -D php-coder/master и потом снова git fetch --force php-coder Надеюсь, что это оно. |
|
1) Вроде получилось: http://prntscr.com/eygwcj, теперь push делать?
2) Вот еще ошибку обнаружил http://prntscr.com/eygwpc, перевода нет
20 апреля 2017 г., 2:27 пользователь Vyacheslav Semushin <
[email protected]> написал:
… Теперь мы нашли причину ambiguous refname -- у тебя есть локальная ветка
php-coder/master которая затеняет такую же но remote (вот он:
http://stackoverflow.com/a/28100234/352708).
Сделай git branch -D php-coder/master и потом снова git fetch --force
php-coder Надеюсь, что это оно.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDAe350EHvapBJhiLPp1Bbo4khXWvvks5rxnxHgaJpZM4MdDEJ>
.
|
Да: git push -f (это для мастера) А потом нужно снова почистить gh571_prediction_country:
и сделать ребейз: git rebase master gh571_prediction_country и потом снова git push -f чтобы запушить эту ветку. |
Это не баг. |
|
Спасибо за терпение! 🥇 (Я спать.) |
|
Создал PR, сейчас буду тестировать, исправил ошибку по которой ссылка не
пропадала.
2017-04-20 2:58 GMT+05:00 Vyacheslav Semushin <[email protected]>:
… Спасибо за терпение! 🥇
(Я спать.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#547 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AVJDAYQOJU9DhjLo9-HRBnuOnK5aNSpuks5rxoOCgaJpZM4MdDEJ>
.
|
Addressed to #517