Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

非アクティブな Dojo を非表示にする仕組みが欲しい (例: CoderDojo 小手指) #310

Closed
yasulab opened this issue Mar 26, 2018 · 9 comments · Fixed by #340
Assignees
Labels
急ぎじゃないよ Make something better but not rushed.

Comments

@yasulab
Copy link
Member

yasulab commented Mar 26, 2018

CoderDojo 小手指の例

CoderDojo 小手指の代表者が入院中で、現在は非アクティブな状態になっている 🏥 😷

あーそうですねぇ。
恐らく年内は再開が厳しそうなので…

「なるべく早く、再開したいと思います!」とのことなので、Dojo の表示/非表示を簡単に切り替える仕組みがあると便利そう? 🤔💭

cf. CoderDojo の運営が難しくなったときのガイドライン
https://coderdojo.jp/docs/how-to-suspend-your-dojo

@yasulab yasulab added the 急ぎじゃないよ Make something better but not rushed. label Mar 26, 2018
@yasulab
Copy link
Member Author

yasulab commented Mar 26, 2018

うーん、DBで削除しようと思ったんですが、そうすると dojo_id で紐づいている他のカラムにも影響が出そうで微妙ですね >< 💦

@yasulab
Copy link
Member Author

yasulab commented Mar 26, 2018

再開の見込みのある休止の場合、status カラムを追加してそこで表示/非表示を切り替えるのがいいのかなぁ🤔💭 cc/ @nalabjp

.oO(統計情報にうまく反映させるようとすると結構な工数がかかりそう 💦だとしたら後回しにしちゃうかなぁ)

@nalabjp
Copy link
Member

nalabjp commented Mar 26, 2018

論理削除やstate的なのは個人的には選びたくない設計なんですが、
手っ取り早いのは手っ取り早いですね。
あとはscopeで特定idを弾くとかも出来るかと思いますが、
defalut_scopeは使いたくないので非表示にしたいトコロ全てでそのscopeを呼ぶってことをやらないといけないですねぇ。(そんなに沢山あるわけでもなさそうなので大した工数ではないかも)

@yasulab
Copy link
Member Author

yasulab commented Mar 26, 2018

コメントありがとうございます! Dojo モデルが has_many ... dependent になっているので、そうするとシンプルに DB から Dojo.find(77).destroy # 77: 小手指 を実行しちゃうのが早いですかね? 🤔💭

has_many :dojo_event_services, dependent: :destroy
has_many :event_histories, dependent: :destroy

@yasulab
Copy link
Member Author

yasulab commented Mar 26, 2018

(正確には実行するだけじゃなくて、dojos.yml と dojo_event_service.yml から id: 77 の部分の削除かな ✂️ )

@nalabjp
Copy link
Member

nalabjp commented Mar 26, 2018

event_historiesは実績としては残したいですよねぇ。
あとは、再開された際にそのまま引き継げると楽といえば楽なので、
個人的には非表示対応の方が良いかなぁと思っていますがどうでしょうか?

@yasulab
Copy link
Member Author

yasulab commented Mar 26, 2018

あとは、再開された際にそのまま引き継げると楽といえば楽なので、
個人的には非表示対応の方が良いかなぁと思っていますがどうでしょうか?

はい、今後のことを考えると非表示対応の方が良いかなと考えています。例えば CoderDojo Zen API とのスムーズなデータ連携ができるようになったら、 Zen 上で非アクティブに変更したら coderdojo.jp からも非表示になる みたいな実装にも繋がって色々と便利にしていける可能性が広がりそうです 💭

ただ今回の小手指の件に関していえば、もともと Google Form で運営されていた関係で統計情報にも大きな影響がないので、「DB上からは削除 && yaml はコメントアウト」という対応でも良いかなと考えています🤔

対応後もこちらの Issue は閉じないで残しておき (例えばアナグラさんのように新しく join する人用のタスクとしてストックしておき) 、別途どこかのタイミングで対応する形でも良いのかなぁとふんわり考えています 😌

@nalabjp
Copy link
Member

nalabjp commented Mar 26, 2018

もともと Google Form で運営されていた関係で統計情報にも大きな影響がないので、「DB上からは削除 && yaml はコメントアウト」という対応でも良いかなと考えています

なるほどなるほど😌

@yasulab
Copy link
Member Author

yasulab commented May 21, 2018

こちらの件、弊社のリソースの関係で「ワークアラウンドな対応として物理削除をやる?」といった話も挙がっていますが、 @naopontan さんにお願いできそうなので、ナラさんの次のアイデアの実現可能性について模索する方向で動いています 🔧 💨 cf. #314

event_historiesは実績としては残したいですよねぇ。
あとは、再開された際にそのまま引き継げると楽といえば楽なので、
個人的には非表示対応の方が良いかなぁと思っていますがどうでしょうか?

@naopontan naopontan self-assigned this Jun 1, 2018
@yasulab yasulab changed the title 非アクティブな Dojo を非表示にする仕組みが欲しい 非アクティブな Dojo を非表示にする仕組みが欲しい (例: CoderDojo 小手指) Jun 5, 2018
naopontan added a commit that referenced this issue Jun 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
急ぎじゃないよ Make something better but not rushed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants