Skip to content
This repository has been archived by the owner on Jun 29, 2020. It is now read-only.

開催予定日が確定していないイベントの一覧が無い #1167

Open
lambdalisue opened this issue Jul 11, 2017 · 7 comments
Open

Comments

@lambdalisue
Copy link
Member

開催予定日が確定してないと一覧とかに表示されないので見失う

@raa0121
Copy link
Contributor

raa0121 commented Jul 12, 2017

普通の一覧にはあえて出さないようにしてるって聞いたことがある気がします。

@miio
Copy link
Member

miio commented Jul 13, 2017

意図としては開催予定日つっこんだまま放置されてつらいって話だったとおもうので、こういう提案はできるかなって。

イベント開催予定日が確定している未来のイベントか、最終更新2週間以内の開催予定日未定のは表示する

実装難易度はちょっとだるそうかもだけど提案だけしてみる。

@giginet
Copy link
Member

giginet commented Jul 13, 2017

意図としては開催予定日つっこんだまま放置されてつらいって話

そのとおり

イベント開催予定日が確定している未来のイベントか、最終更新2週間以内の開催予定日未定のは表示する

これが一番簡単にできそうかな

@miio
Copy link
Member

miio commented Jul 13, 2017

意図としては開催予定日つっこんだまま

ちょっと日本語おかしかったw 開催予定日未定でつっこんだまま ですねw

@giginet
Copy link
Member

giginet commented Jul 13, 2017

指定されたユーザーに公開されたイベントの中で、まだ終わっていない
or イベントの終了時期が指定されていないかつ、作成日から30日日以内のイベントを含むクエリを返す

あれー?今もそれっぽいことしてるぞ!

https://github.com/kawazrepos/Kawaz3rd/blob/develop/src/kawaz/apps/events/models.py#L38

開始日未定については考慮されてないのかな。開始日未定かつ作成日からn日以内もさらに追加すると良さそう

@giginet
Copy link
Member

giginet commented Jul 13, 2017

わかった。お前が全て悪い

https://github.com/kawazrepos/Kawaz3rd/blob/develop/src/kawaz/apps/events/filters.py#L14

イベント開催予定日が確定している未来のイベントか、作成日30日以内の開催予定日未定のを取ってくる仕組みはあったが、表示するときに使われていなかったと言うことかな?

@lambdalisue
Copy link
Member Author

違う

https://github.com/kawazrepos/Kawaz3rd/blob/develop/src/kawaz/apps/events/models.py#L43-L44

これ andor 使ってるけど、これじゃ bool 値としてこの時点で評価される。
正しくは少し下にあるように &| を使う。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants