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

🐛: santoku-appのgoogle-services.dummy.jsonをそのまま使えるように修正 #355

Merged

Conversation

nagano-ryo
Copy link

✅ What's done

  • santoku-appのandroid/app/google-sevices.dummy.jsonのpackage_nameをdummyからjp.fintan.mobile.santokuappjp.fintan.mobile.santokuapp.debugに修正

Tests

  • android/app/google-sevices.dummy.jsonandroid/app/google-sevices.jsonにコピーし、ファイル修正することなくアプリが起動すること

Devices

  • 動作確認に利用したデバイスにチェックをつけてください
  • Android
    • エミュレータ (Pixel 3a/Android 11)
    • 実機 (Pixel 3a/Android 11)

Other (messages to reviewers, concerns, etc.)

なし

@nagano-ryo nagano-ryo enabled auto-merge (squash) July 13, 2021 09:49
@in-a-paddy in-a-paddy changed the title 🐛: ユーザーの手間を考慮し、package_name値をdummyからjp.fintan.mobile.santokuappに修正 🐛: santoku-appのgoogle-services.dummy.jsonをそのまま使えるように修正 Jul 13, 2021
@in-a-paddy in-a-paddy disabled auto-merge July 13, 2021 10:07
@in-a-paddy in-a-paddy enabled auto-merge (squash) July 13, 2021 10:08
@in-a-paddy in-a-paddy merged commit c03c2aa into master Jul 14, 2021
@in-a-paddy in-a-paddy deleted the fix/santoku-app-google-services-json-dummy-package-name branch July 14, 2021 02:27
creature-water-valley added a commit that referenced this pull request May 8, 2023
## ✅ What's done

[`rn-spoiler`のExpo SDK 48アップグレード対応](https://github.com/ws-4020/rn-spoiler/pull/133)で以下を検知

ToDo一覧画面のFAB(Icon)は、`react-native-elements`の`Icon`を使用していますが、`raised`を指定した場合に以下のWarningが出力されます。

> WARN  (ADVICE) View #355 of type RCTView has a shadow set but cannot calculate shadow efficiently. Consider setting a background color to fix this, or apply the shadow to a more specific component.

`shadow`関連のPropsを扱う場合は`backgroundColor`も設定した方が良いという警告ですが、`raised`を`true`に設定した場合に、`react-native-elements`では`backgroundColor`を設定していないようです。

- https://github.com/react-native-elements/react-native-elements/blob/v3.4.3/src/icons/Icon.tsx#L177

ToDoアプリでは、`Icon`の`raised`の使用を学習の目的としていないので、ToDo一覧画面のFAB(Icon)から`raised`を削除しました。

- [x] `raised`の削除
  - 以下で対象を確認(`shadow`系のPropsを使用していないかも併せて確認)
  - `git grep -E "raised|shadow[Color|Offset|Opacity|Radius]" -- website/`
- [x] ToDo一覧画面のキャプチャを貼り付ける箇所のキャプチャ更新
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.

3 participants