Skip to content

期間を指定する部分を改善した#7

Open
moba1 wants to merge 1 commit intohinananoha:mainfrom
moba1:feature/fix-period-option
Open

期間を指定する部分を改善した#7
moba1 wants to merge 1 commit intohinananoha:mainfrom
moba1:feature/fix-period-option

Conversation

@moba1
Copy link
Copy Markdown
Contributor

@moba1 moba1 commented Nov 24, 2021

目的

期間指定に -c-r という2つのオプションが存在してるのを統一したい

詳細

統一するにあたって、 -c-r を同一に扱えるフォーマットが必要となる。
そこで、 -r に JSON を指定できるようにして、これらを切り替えられるようにした。
現在は今月を意味する {"type": "current-month" } とある期間を表す {"type": "period", "begin": "YYYY-mm-dd", "end": "YYYY-mm-dd"} の2つのフォーマットを提供している。
つまり、 type プロパティに指定した種類によって JSON の内容を切り替えることで統一的にかつ柔軟に期間指定ができるようになった。

period タイプの beginend の指定はオプショナルとしてある。
これによって「2020/12/10 以降」を抽出したり、「2020/10/01」までのデータを抽出できるようになった。

注意事項

これを含んだものをリリースする際は、後方互換性がなくなっているので v2.0.0 にバンプさせる必要がある

@moba1 moba1 force-pushed the feature/fix-period-option branch from acc8c9c to b339030 Compare November 24, 2021 15:03
@hinananoha
Copy link
Copy Markdown
Owner

hinananoha commented Nov 25, 2021

これを含んだものをリリースする際は、後方互換性がなくなっているので v2.0.0 にバンプさせる必要がある

同様の理解。これに関しては、現在main関数でCSV出力までしているのを、CSV出力を別関数に分離して、モジュールとして扱いやすくしたい気持ちがあり、そこと併せて実装をしたい。

また、 -c オプション自体にも一定の需要はあると考えており(今月の注文を迅速に一覧化したいという意味で)、同時にperiodのbegin/endがoptionlにすることで「~以降」や「~以前」を表現するのも実装したいので、そういった点も考えつつ他の実装と併せて修正を行いたい。

@moba1
Copy link
Copy Markdown
Contributor Author

moba1 commented Nov 25, 2021

その場合、おそらく git のようにサブコマンドを用意しておくのがよいのではないだろうか?
たとえば、

# 今月分の表示
booth_order_list.py current-month
## エイリアス
booth_order_list.py cu

# 特定期間の表示(`--begin` か `--end` のいずれかを指定するように義務付ける)
booth_order_list.py range --begin YYYY-mm-dd --end YYYY-mm-dd

# 指定なし
booth_order_list.py non-range
## エイリアス
booth_order_list.py nr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants