キャラクター選択可能なウィジェットを追加

Koshimizu-Takehito commented 3 years ago

概要

キャラクター選択可能な small ウィジェットを試しに追加してみました。好みあると思いますので、そっとクローズしていただいてもOKです👍

既存 small, medium ウィジェットのように時間でキャラクターを更新しつつ、キャラクターを選択する機能は実装していません。「ウィジェットを編集」の選択値をプログラムから指定できなそうなことが理由で諦めました。

やったこと

実装内容

Making a Configurable Widget にしたがって実装
- SiriKit intent definition を追加
- Intent Handler を追加
- Intent Timeline Provider を実装

プロジェクト構成（概要）

UhooiPicBookWidgetsConfigurableIntent ターゲットを新規追加

.
├── UhooiPicBookWidgets
│    └── Monster
│        └── MonsterConfigurableWidget.swift        # Configurable Widget / Intent Timeline Provider
└── UhooiPicBookWidgetsConfigurableIntent
     ├── IntentDefinition                           # Intent Definition / Intents files
     └── IntentHandler.swift                        # Intent Handler

スクリーンショット

ウィジェットの追加

新規の small を追加

ウィジェットの編集

1.選択前	2.編集	3.一覧表示	4.選択後

uhooi commented 3 years ago

@Koshimizu-Takehito PRありがとうございますー！私の実装漏れで申し訳ないのですが、 UhooiPicBookWidgets フォルダがSwiftLintの適用外になっていたので、 develop ブランチを取り込んでプッシュしていただけると助かります 🙇 SwiftLintの警告は、直しづらかったらそのままでも大丈夫です！

Koshimizu-Takehito commented 3 years ago

@uhooi 了解です！対応しますー！

Koshimizu-Takehito commented 3 years ago

@uhooi cecde4b で develop をマージしました。今回の実装範囲内では Lint 警告なしでした 👍 c47e27c で SwiftLint の適用範囲を追加しました。よろしくお願いしますー！

uhooi commented 3 years ago

ご対応ありがとうございます 🙏 レビューはウィジェットについて勉強しつつ行いたいため、もう少しかかりそうです 🙇

Koshimizu-Takehito commented 3 years ago

承知しましたー！勉強教材として、この PR を使っちゃってください ☺️

uhooi commented 3 years ago

まだウィジェットの勉強不足でレビュー中ですが、取り急ぎ気になる箇所だけコメントしました！デバッグして動作は問題なさそうでした、好きなキャラクターを選べるの最高過ぎる…笑

uhooi commented 3 years ago

@Koshimizu-Takehito レビューが完了しました！コメントが多くて恐縮ですが、よろしくお願いします 🙇

Koshimizu-Takehito commented 3 years ago

@uhooi 丁寧にフィードバックしていただきありがとうございます ☺️ 時間を見つけて諸々修正いたします 🙏

Koshimizu-Takehito commented 3 years ago

修正ひきつづきやっていきます！終わりましたらメンション飛ばしますので、よろしくお願いします！

Koshimizu-Takehito commented 3 years ago

@uhooi 修正終わりました。ご確認よろしくお願いします 🙏

uhooi commented 3 years ago

@Koshimizu-Takehito ご修正ありがとうございます…！いくつかコメントしたのでご確認お願いします（何度もすみません、、）。

Koshimizu-Takehito commented 3 years ago

@uhooi 何度も見ていただいてありがとうございます！！ 🙇 修正とコメント返信しましたので、ご確認よろしくお願いします。

uhooi commented 3 years ago

LGTMです 👍 本当にありがとうございました！！！

Koshimizu-Takehito commented 3 years ago

レビューありがとうございましたー！

uhooi / UhooiPicBook