pj-picbook / picbook

9 stars 0 forks source link

Feature/#137 book stamp #148

Closed Romu1273x closed 1 year ago

Romu1273x commented 2 years ago

関連のタスクissue*

PR作成時のチェックリスト*

※対象外の場合は実施していなくてもチェックを入れる

対応したこと*

変更前のキャプチャ

変更後のキャプチャ

simulator_screenshot_18AD3D83-753B-4DB5-A8C0-9C3E1A9FD5BC

動きがある場合(GIF動画など)

※ ヒント : 入力エリアに画像をドラッグ&ドロップするとアップロードできる -->

未解決の課題

その他・備考

リリース時の注意点

karokita commented 1 year ago

@Romu1273x 月の切り替えもサクサク動いて、いい感じのUIになってますね。ありがとうございます!

本の登録ボタンを押したタイミングで、スタンプがどうにもつかずだったので、 後ほどのMTGで詳細伺えるとありがたいです🙏

Romu1273x commented 1 year ago

@karokita すみません、スワイプで更新するように修正しました。

Romu1273x commented 1 year ago

スワイプの難易度が高かったので何もしなくても更新されるように変更しました。 ただ、関数が常にコールされるようになってるのでどこかのタイミングで直した方がいいかもです

Romu1273x commented 1 year ago

@tokku5552 レビューありがとうございます! 指摘対応いたしました。

karokita commented 1 year ago

@Romu1273x
issueいったりきたりですみません笑

完全にやま都合ではあるのですが、 

137 を含めて無事ストアにひっかからずにアップできるかを試したい

&発表会資料用に動画を撮りたいという事情がありまして、

可能であれば、 現時点の対応をもって一度マージいただき、 ここのやりとりについては、徳田さんのレスに応じて別issue立てて進行という形でいけると嬉しいのですがいかがでしょうか?🙏

Romu1273x commented 1 year ago

@karokita @tokku5552 では、取り急ぎでマージしてしまいます。

karokita commented 1 year ago

@Romu1273x マージありがとうございます!今、シミュレーターと実機で見てたのですが、アプリをタスクキルして再起動してもスタンプがつかなくなっているような模様でして、お手すきでご確認いただけるとありがたいです🙏

Romu1273x commented 1 year ago

@karokita こっちでdevelopとマージ前のブランチ両方とも試して見ましたが、アプリ再起動しなくてもちゃんと動いてました。

データの問題のような気がするのですが、その辺の確認はされましたか?

karokita commented 1 year ago

@Romu1273x おお!Romuさんの環境だといけてるんですね。

・最新のdevelopブランチでビルド ・新しく書籍登録 ・マイページで冊数カウントされてることは確認 ・スタンプページにアクセス(再起動後もチャレンジ)

という感じの確認状況でした。 今外に出てしまったので、また改めて確認してみたいと思いますー!✋

karokita commented 1 year ago

(途中経過のつぶやきメモ) キャッシュクリアしたり色々試してみたのですが、今のところきびしそうです。

Romuさん環境との差分を考えてみたのですが、 バンドルIDを新しいものに変えたんですが、それ関係あったりするんでしょうか...

明日、みてみようと思います🤔

karokita commented 1 year ago

新しくアカウント作り直したらスタンプつきました!わいわい アカウント作成のタイミングが影響しているのかも

Romu1273x commented 1 year ago

了解しました。 MTGの時にやまさん、bookshelfsコレクション消してたからそれが原因だと思います。