search

eshcherbin / save-the-moment

SPbAU educational Android project
0 stars 0 forks source link

Added converting position to address #20

Closed eshcherbin closed 7 years ago

eshcherbin commented 7 years ago

Теперь при выборе локации автоматически подгружается адрес.

krinkinmu commented 7 years ago

2016-12-04 12:25 GMT+03:00 Egor Shcherbin notifications@github.com:

@eshcherbin commented on this pull request.

In app/src/debug/res/values/google_api_keys.xml https://github.com/eshcherbin/save-the-moment/pull/20:

@@ -1,3 +0,0 @@ -

  • AIzaSyDjRyTIqsevq_R0WT54Ny6gNCzQg8KtQnI

Хотелось, чтобы в origin лежал файл, в котором вместо ключа написано YOUR_KEY_HERE, а локально хранился реальный ключ, так как выкладывать ключ в открытый доступ нежелательно. К сожалению, видимо, я ещё не справился с настройками, поэтому то ли git, то ли Android Studio упорно игнорируют мой .gitignore и добавляют этот файл в коммиты.

Как раз в ближайшее время собираюсь решить эту проблему.

Не очень понятно, если этот ключ для отладочной версии, то не понятно почему его не желательно выкладывать на github, едиснтвенная причина не делать этого это если вы пользуетесь несколькими ключами и они будут мешать друг другу.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/eshcherbin/save-the-moment/pull/20, or mute the thread https://github.com/notifications/unsubscribe-auth/ABH4c95cCRcBclLpQYl10Web102AZYPgks5rEoclgaJpZM4LDc1V .

eshcherbin commented 7 years ago

Хорошо, пусть будет в открытом доступе.

krinkinmu commented 7 years ago

Ок, но только я так и не понял, вы пользуетесь одним ключем или нет, и если ключей несколько почему бы не генерировать просто сам Manifest (кажется так делают, если ключей несколько)?

eshcherbin commented 7 years ago

В каком смысле генерировать? Вставлять ключи прямо в манифест? Да, пока ключ один, возможно, ещё появится от Google Drive.

krinkinmu commented 7 years ago

2016-12-04 13:42 GMT+03:00 Egor Shcherbin notifications@github.com:

В каком смысле генерировать?

ну например генерировать Manifest.xml при скачивании репозитория или как-то так (пользуясь git hook-ами)

Вставлять ключи прямо в манифест?

ну да, можно конечно выносить его и в resources, но ключ не очень выглядит как ресурс.

Да, пока ключ один, возможно, ещё появится от Google Drive.

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

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/eshcherbin/save-the-moment/pull/20#issuecomment-264696366, or mute the thread https://github.com/notifications/unsubscribe-auth/ABH4cwXFtIn6uLiXcm5s2pI5q9-ZdChEks5rEpkogaJpZM4LDc1V .

eshcherbin commented 7 years ago

ну да, можно конечно выносить его и в resources, но ключ не очень выглядит как ресурс.

Хм, при создании шаблонной Maps Activity файл для ключа создаётся в ресурсах ровно таким образом.

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

В этом смысле ключ тоже один. Не уверен, что есть большой смысл делать больше.

В любом случае, ключи не так сильно связаны с reverse geocoding. Я мерджу этот PR?

krinkinmu commented 7 years ago

Ок.

2016-12-04 14:04 GMT+03:00 Egor Shcherbin notifications@github.com:

ну да, можно конечно выносить его и в resources, но ключ не очень выглядит как ресурс.

Хм, при создании шаблонной Maps Activity файл для ключа создаётся в ресурсах ровно таким образом.

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

В этом смысле ключ тоже один. Не уверен, что есть большой смысл делать больше.

В любом случае, ключи не так сильно связаны с reverse geocoding. Я мерджу этот PR?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/eshcherbin/save-the-moment/pull/20#issuecomment-264697276, or mute the thread https://github.com/notifications/unsubscribe-auth/ABH4c2G4jnk6vEMoW9cYNkvkd4An-hZoks5rEp4tgaJpZM4LDc1V .