Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Локализация ответов #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Seg-mel
Copy link

@Seg-mel Seg-mel commented Mar 8, 2018

#6
README думаю вам самим нужно будет дописать в том формате, в котором именно вы хотите

@igor-shevchenko
Copy link
Member

Андрей, спасибо! Документацию допишу. Есть пара вопросов:

  1. Точно ли по умолчанию используется локаль ru-RU? В документации написано только про русский язык, а время, насколько я помню, приходит в UTC, а не MSK. Если поведение без локали и с ru-RU чем-то отличается, то лучше будет оставить клиент обратно совместимым и по умолчанию не передавать никакую локаль.
  2. Мы парсим и формируем даты, предполагая, что они в UTC. Нормально ли это будет работать, если сервис будет присылать и ожидать даты в часовом поясе, заданном локалью?

@Seg-mel
Copy link
Author

Seg-mel commented Mar 9, 2018

@igor-shevchenko

  1. Я просто проверил какие ответы приходят сейчас, поэтому русский и выбрал. Изучу этот вопрос.
  2. Тут надо обратиться к разработчикам CloudPayments, потому что, как оказалось, они не все локализируют. Например города и области всегда русские! Тут надо будет уточнять что же они локализируют, а что нет. Ну или заставить локализировать все, а то для меня это выглядит очень странно=)

Чуть позже займусь этим.

@igor-shevchenko
Copy link
Member

igor-shevchenko commented Mar 20, 2018

Пообщался с поддержкой по поводу локализации, узнал следующее:

  1. Часовые пояса указаны в документации просто так, при смене локализации время нигде не меняется.
  2. Переводиться будет только поле CardHolderMessage, остальные (в том числе и данные о местоположении) останутся на исходном языке.
  3. Сейчас перевод не работает ¯_(ツ)_/¯ Будут устранять эту проблему.
  4. Возможно, коды языков еще поменяются. Это будет известно, когда починят перевод.

В связи с этим пока что оставлю пул-реквест открытым. Когда на стороне сервиса всё поправят, я протестирую, смёржу и доделаю то, что будет нужно.

@Seg-mel
Copy link
Author

Seg-mel commented Mar 20, 2018

@igor-shevchenko у меня тут времени не было, но я на той неделе тоже с ними пообщался, кучу предложений отправил, потому что много косяков, но переводы у меня работали. Просто они не все переводят. Вот что мне ответили:

  1. Переводятся только сообщения которые показываются пользователю
  2. Переводить остальное не собираются

@igor-shevchenko
Copy link
Member

@Seg-mel а на каком методе можно проверить, что локализация работает? В тех запросах, которые я делал, CardHolderMessage не менялся, и как мне сообщили, пока что так и должно быть, но скоро починят.

@igor-shevchenko igor-shevchenko mentioned this pull request Mar 22, 2018
@igor-shevchenko
Copy link
Member

Никакой новой информации не появилось? У меня по-прежнему ничего не работает.

@Seg-mel
Copy link
Author

Seg-mel commented Apr 16, 2018

@igor-shevchenko сегодня проверю еще раз, уберу дефолтное значение. Что-то работы навалилось, и я запамятовал, что тут не доделал =) отпишусь как что-то сделаю.

@igor-shevchenko
Copy link
Member

Пока что непонятно, работает ли локализация в принципе. Если не работает, я бы предложил ничего пока не делать. Потому что потом она может заработать как-то не так, как мы ожидали, и всё сломается.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants