search

ftnext / python-as-pyconjp-staff

PyCon JPのスタッフをPythonにもやらせよう
MIT License
6 stars 2 forks source link

[feat] Create a function to get list of meetings in future #48

Closed peacock0803sz closed 3 years ago

peacock0803sz commented 3 years ago

ミーティングの一覧を返す関数を作成

全体の設計方針読めなかったので、最小限の機能を追加しておきました。あとは煮るなり焼くなりどうぞ!

list_meetings 関数の説明

取ってきたuser_idを渡すとそのユーザーが見られるミーティングの一覧を返す

def list_meetings(user_id: str) -> list[dict[str, str]]:

テスト用notebook: https://gist.github.com/peacock0803sz/b86022573cde22b89e6746914f767a35 使うにはtouch mtg_secretary/__init__.pyする必要あり

ftnext commented 3 years ago

プルリクエストありがとうございました。 こちらはcloseします。

listサブコマンドは、このプルリクエストが出される数日前から #44 （と対応ブランチ）にて実装中でした。 ref: https://github.com/ftnext/python-as-pyconjp-staff/pull/57/commits/ddd842b7d8bed3cc16bd001e0e9ef2280c7f4ac3

上記コミットの時点でスクリプトが見通しづらくなってきたので、独自定義したクラスを使ったリファクタリングに踏み切りました。 datetimeまわりでハマったため、クラスの導入の完遂を後回しにしていましたが、本日ようやく終わりmasterにマージしました。 これにより listサブコマンド が使えるようになりました。

$ python schedule_zoom_mtg.py list

これまでの機能はcreateサブコマンドで使えます