全体的なフローの修正

mollinaca / flyers2

Creative Commons Zero v1.0 Universal

0 stars 0 forks source link

Open mollinaca opened 2 years ago

mollinaca commented 2 years ago

現状、ちゃんと動いてれば問題ないけど各処理で問題が発生したときに脆いのでいろいろ直す・最初に「現在取得できるチラシリスト」を作る・last.jsonと比較して、実際に取得するチラシリストを作る・それをもとに実際に取得してPOSTする、POSTできたらlast.jsonを更新する

ファイルのオープン・クローズが格段に増えると思うけど、別に気にならんだろうと思うのでこういう方針にする

mollinaca commented 2 years ago

・files.upload に失敗したときのワークアラウンド

mollinaca commented 2 years ago

・shop_url から png/pdf のurlを取得する・last.jsonと比較（last.jsonの flyers:[] の中身はURLの一覧になる）・あればpass ・なければ取得してPOST ・新しい last.json へ書き込み、ファイルwrite

URLの数だけファイルのopen/closeになるが、都度やったほうが良さそうなのでそうする

mollinaca commented 2 years ago

・チラシファイルをPOSTするときに店名を入れる店名は、config.ini でURLといっしょに渡す

mollinaca commented 2 years ago

メモ

・files.upload に失敗したときのワークアラウンド

これは、slack-sdk が、サーバから非jsonなレスポンスを受けた時点で例外を投げるので、slack-sdkをやめたほうがよさそううーむ、ステータスコードとかでハンドリングできればいいんだけど、それを考えるなら結局WebAPIを使って自分で実装したほうが早いか。