のえる DTP鯖管 @noellabo@dtp-mstdn.jp

@mayaeh @tateisu 正常系のパスをif〜endの外に出した方がわかりやすい、とかはあるかもしれないなぁ。ifで判定するブロックは全部エラー判定（before_action的な）にするとか。

などという、特に結果に寄与しないコード談義も楽しいのであります。戦争が起きる系ですがw

風の谷のガンシップみたいだな、とか思った……。

@mayaeh @tateisu あっさりマージしてくれましたね。

私は、最初に提案した書き方が、シンプルで重複がなく、ミスが混入しにくい点で優位と考えています。判定を追加するのもシンプルに対応できるし。正常判定のパスは先頭か末尾のいずれかであればどちらでもいいんじゃないかな。

ただ、独学というか、rubyに至ってはまともに文法勉強会してないので(!)ガイドラインなどがあるのかどうか知らなかったり……。

ちゃんと目を通したのはcrystalのリファレンスだけw

いまだに住基カードを身分証として愛用している。電子証明書も入れられるが、入れてから使わなかったので失効したままほったらかし。

@mayaeh @tateisu
この記法もアリかな。

def require_user!
if current_user && !current_user.disabled? && current_user.confirmed?
set_user_activity
elsif current_user && current_user.disabled?
render json: { error: 'Your login is currently disabled' }, status: 403
elsif current_user && !current_user.confirmed?
render json: { error: 'Email confirmation is not completed' }, status: 403
else
render json: { error: 'This method requires an authenticated user' }, status: 422
end
end

あら（私的）大ニュースだわこれ
https://twitter.com/alice_soft/status/1077501336709709824

@rat@mstdn.rabbitodon.work railsとか、react + reduxとか、フレームワークをガッツリ理解していないと手を出しにくい構造になってるの、私も正直ツライです。

MODXのように機能にしてもコンテンツにしても追加しやすい（まぁCMSとしては当たり前なんですが）ものと比較すると、もの凄く開きがあるのを実感しています。

Mastodonにこだわる必要がないというのは、それはそうなんだけど、今はMastodonをとことんよくしていくことが、Fediverseに貢献すると思うんだよね。

@tateisu @mayaeh こちらの件、
https://mastodon.juggler.jp/@tateisu/101301391429326074

とりあえずそのままプルリク投げました。
https://github.com/tootsuite/mastodon/pull/9625

@tateisu @mayaeh まぁそうですね……。HTTPとは別に、クライアントとの連携のためのエラーコードとかあってもいいよなーとか（いよいよいまさら

@mayaeh @tateisu 条件判断が重複しない、処理がシンプルな書き方に直しちゃいましたが、全体のコードをみてないので、すりあわせした方がいいかもしれませんね。

児童プログラミング……幼女戦記的な？

@mamemomonga 今からやってもいいのでは！
あるいは、〜海のすがた、とか！

@mamemomonga www

でも、ドラクエの海の魔物達が現れるイメージはあります！

@mamemomonga おお、海上だ！ ロマンがあるなぁ。

@tateisu @mayaeh 違うメッセージを出すようにはしたけど、アプリから判別して処理を分けるのに必要な情報にはなっていないよね。翻訳されるだろうし。

403を返すの一緒だし。

何を返したらいいんだろ？

@tateisu @mayaeh こんな感じで。
https://github.com/dtp-mstdn-jp/mastodon/pull/194/commits/fbd57f3a0e19aaf83edb705d83fb3b0e41315f5f

curl -X GET -H "Authorization: Bearer $ACCESS_TOKEN" -sS $INSTANCE/api/v1/accounts/verify_credentials

{"error":"Email confirmation is not completed"}