smknhd-tech / english-class-notify Goto Github PK

PythonNotify....これは酷い

Originally posted by @KAZYPinkSaurus in #27

ログイン必要だから面倒かなぁ
じんこうちのーが賢く予約してほしい

まずは、どんなデータを溜めたいのか、から。

• データの種類
• データの質
• データの形式
• データを使ってやりたいこと

dockerfileのベストプラクティスに則って、空のディレクトリに置きます。
http://docs.docker.jp/engine/articles/dockerfile_best-practice.html#dockerignore

今後、DBコンテナ用のdockerfileなどが増えてくると思うので、将来を想定したディレクトリ構成を考えます。

DBのデータを用いて曜日時間ごとのクラスのオープン状況の可視化

画像作ってAPIに投げてあげましょう

e.g LINEで teacherA schedule とメッセージすると予定を返してくれる bot にならんかな
(できるかは知らない)

あるとうれしいよねぇー

スクリプト実行時に、↓で新規追加したテーブルに、データを登録できるようにします。
#9

if message の message をなくして 予約なかったら 現在予約なしなどと PythonNotify関数でつぶやいてもらうこと出来る?

Originally posted by @KAZYPinkSaurus in #16

unittestがシンプルで良いのではないでしょうか。
https://blog.amedama.jp/entry/python-unittest

どうせなら、dockerのことも考慮した構成にしましょう。
https://www.rhoboro.com/2018/09/07/docker-exclude-tests.html

スケージュールには ｢空欄｣, ｢終了｣, ｢予約済｣, ｢予約可｣の4状態がある｡
記録することで講師の行動がわかる｡
スケジュールは24スロット*8日=192スロット

例えば,
空欄→予約可 は講師がクラスをオープンしたことを意味する
予約可→空欄 は講師がクラスをクローズしたことを意味する

なにをしたではなく、なぜそうしたかを書く。処理の内容ではなく、処理の目的。

詳細なDB構成の検討は後回し。とりあえずファイルを脱却し、DBへ。

ちゃんと考えないと🚓👮‍♀️

これをやめたい

https://github.com/KAZYPinkSaurus/kazy-english-class-notify/blob/872edc098a44c53560c891acb8ad77a6f5e05378/notify.py#L30

if day == "" or day == "today":
        days = 0
    elif day == "tomorrow":
        days = 1
    else:
        days = int(day)

現在ソースにコーディングされている

差分があった時だけ通知したいーー

予約可じゃない状態から、予約可になった時のみ通知が来るようにしたい。

アプリケーションはアプリケーションでテストをし（#22）、こちらはdocker ビルドがちゃんとできてるのかなどをテストします。
https://qiita.com/ko1nksm/items/91932b1d4f59a57d88ea