マップの表示 #26

kirikirisu · 2019-12-16T10:27:04Z

概要

Issueのタスクを完了させる

対応するIssue

#4

kirikirisu · 2019-12-16T10:38:18Z

デバックを行う際はAPI_KEYという環境変数にgoogleのapiキーを設定してください。
apiキーはslackの2019_kanazawa_app_gitにあります。
環境変数の設定の仕方はWikiにあります。

tsuru-kazu · 2019-12-17T08:10:20Z

app/src/main/res/values/strings.xml

@@ -1,3 +1,4 @@
 <resources>
    <string name="app_name">kanazawaApp-2019</string>


キャメルケースになっていないよ。
app_name -> appName
に変更しよう！

了解しました

tsuru-kazu · 2019-12-17T08:11:22Z

app/src/main/java/com/example/kanazawaapp_2019/ShelterActivity.kt

+     * installed Google Play services and returned to the app.
+     */
+    override fun onMapReady(googleMap: GoogleMap) {
+        mMap = googleMap


なぜmMapという変数名にしたんですか？
個人的にこれだとgMapの方が分かりやすい。

デフォルトの変数名をそのまま使いました。
gMapの方が確かに分かりやすいと思ったため次のコミットで変えときます。

IrukNuj · 2019-12-18T03:12:50Z

ぱっと見良さそうです。
他の人が見たときのために画面のスクショをPRに貼ってくれると嬉しいかもです
（作業中でその段階じゃなかったらごめんなさい 🙇

IrukNuj · 2019-12-18T03:11:12Z

app/src/main/AndroidManifest.xml

+        -->
+        <meta-data
+            android:name="com.google.android.geo.API_KEY"
+            android:value="${API_KEY}" />


API_KEY管理ちゃんとできてるのgoodです 👍

… into createScreen-shelter-kirisawa

kirikirisu · 2020-01-02T01:41:01Z

機能は全て実装できました。

kirikirisu · 2020-01-02T01:50:51Z

初回マップ起動時にローケーションのパーミッション確認があります。認証をしてから、現在地ボタンを押して現在地に移動してください。
次からの起動時は、金沢市を中心にマップが起動し、現在地ボタンを押すとエミュレータで設定した現在地に戻ります。

shxun6934 · 2020-01-08T09:24:54Z

.gitignore

@@ -24,4 +24,4 @@ gradlew
 gradlew.bat
 local.properties
 /build
-*.apk
+*.apk


なんの差分？

あんまり意味なさげだから，上げない方がいいかも

了解しました

shxun6934 · 2020-01-08T09:25:00Z

app/.gitignore

@@ -1 +1 @@
-/build
+/build


shxun6934 · 2020-01-08T09:26:05Z

app/build.gradle

@@ -14,6 +14,7 @@ android {
        versionCode 1
        versionName "1.0"
        testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
+        manifestPlaceholders = [API_KEY: System.getenv("API_KEY")]


APIKEYを隠すのはGood!!

shxun6934 · 2020-01-08T09:31:42Z

app/src/main/java/com/example/kanazawaapp_2019/ShelterActivity.kt

+        if (ContextCompat.checkSelfPermission(this,
+                android.Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED) {


上でpermission確認しているのに，ここでもやる意味は？

これがないとなぜかクラッシュしてしまいます

なるほど

shxun6934 · 2020-01-08T10:45:14Z

app/src/main/java/com/example/kanazawaapp_2019/ShelterActivity.kt

+        fetchCsv("shisetsu_hinan.csv")
+
+        /*パーミッションがあるか確認*/
+        if(ContextCompat.checkSelfPermission(this, android.Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED){


こっちもクラッシュする？

初回マップ表示時（GPSの使用許可がない状態）で現在地を取得しようとすると、エラーで起動できません。
そのため、どうしてもここで一度確認をしなければなりませんでした。

この前に現在地取得のメソッド走らせたらどー？

マップの表示

cb97efb

kirikirisu self-assigned this Dec 16, 2019

kirikirisu added レビュー依頼作業中 labels Dec 16, 2019

kirikirisu removed the レビュー依頼 label Dec 16, 2019

tsuru-kazu reviewed Dec 17, 2019

View reviewed changes

IrukNuj reviewed Dec 18, 2019

View reviewed changes

kirikirisu added 5 commits December 18, 2019 18:31

csvの追加

0df851d

Merge branch 'master' of https://github.com/kirikirisu/kanazawaApp-2019…

511b5fe

… into createScreen-shelter-kirisawa

csv読み取り完了

efb64f7

mapに観光地のピンを表示

e03bc41

現在地の取得

10de68a

kirikirisu added the レビュー依頼 label Jan 2, 2020

現在地まで戻るボタンの実装

330b9f3

kirikirisu added 4 commits January 2, 2020 11:27

バグの修正

c4f6472

初回マップ画面訪問時の挙動修正

229fd6a

toolBarの実装

591a39f

変数名の変更

bf7af24

kirikirisu removed the 作業中 label Jan 7, 2020

shxun6934 reviewed Jan 8, 2020

View reviewed changes

app/.gitignore

@@ -1 +1 @@

/build

/build

Copy link

Collaborator

shxun6934 Jan 8, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

同上

shxun6934 reviewed Jan 8, 2020

View reviewed changes

shxun6934 added reviewed Return review and removed レビュー依頼 labels Jan 8, 2020

コンフリクト解消

25be5b8

shxun6934 reviewed Jan 8, 2020

View reviewed changes

kirikirisu added the レビュー依頼 label Jan 14, 2020

kirikirisu added 2 commits January 29, 2020 19:02

faild

a062bed

deleteMaterialUi

e380bc0

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

マップの表示 #26

マップの表示 #26

kirikirisu commented Dec 16, 2019 •

edited

Loading

kirikirisu commented Dec 16, 2019 •

edited

Loading

tsuru-kazu Dec 17, 2019

kirikirisu Dec 17, 2019

tsuru-kazu Dec 17, 2019

kirikirisu Dec 17, 2019

IrukNuj commented Dec 18, 2019 •

edited

Loading

IrukNuj Dec 18, 2019

kirikirisu commented Jan 2, 2020

kirikirisu commented Jan 2, 2020 •

edited

Loading

shxun6934 Jan 8, 2020

shxun6934 Jan 8, 2020

kirikirisu Jan 8, 2020

shxun6934 Jan 8, 2020

shxun6934 Jan 8, 2020

shxun6934 Jan 8, 2020

kirikirisu Jan 8, 2020

shxun6934 Jan 8, 2020

shxun6934 Jan 8, 2020

kirikirisu Jan 8, 2020

shxun6934 Jan 15, 2020

shxun6934 Jan 15, 2020

		@@ -1,3 +1,4 @@
		<resources>
		<string name="app_name">kanazawaApp-2019</string>

		if (ContextCompat.checkSelfPermission(this,
		android.Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED) {

マップの表示 #26

Are you sure you want to change the base?

マップの表示 #26

Conversation

kirikirisu commented Dec 16, 2019 • edited Loading

概要

対応するIssue

kirikirisu commented Dec 16, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

IrukNuj commented Dec 18, 2019 • edited Loading

Choose a reason for hiding this comment

kirikirisu commented Jan 2, 2020

kirikirisu commented Jan 2, 2020 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kirikirisu commented Dec 16, 2019 •

edited

Loading

kirikirisu commented Dec 16, 2019 •

edited

Loading

IrukNuj commented Dec 18, 2019 •

edited

Loading

kirikirisu commented Jan 2, 2020 •

edited

Loading