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

ログイン必須の指定が多すぎる #5984

Closed
komagata opened this issue Dec 27, 2022 · 1 comment
Closed

ログイン必須の指定が多すぎる #5984

komagata opened this issue Dec 27, 2022 · 1 comment

Comments

@komagata
Copy link
Member

komagata commented Dec 27, 2022

controllerにbefore_action :required_loginと書いているが、ほとんどのページがログイン必須なので、デフォルトをログイン必須にして、ログインがいらないページにだけskip_before_actionした方が漏れが少ない。
影響ページが多いのでテストをたくさん書く必要がありそう。

@komagata komagata created this issue from a note in 開発 (やる) Dec 27, 2022
@komagata komagata moved this from やる to いつかやる in 開発 Dec 27, 2022
@komagata komagata moved this from いつかやる to 今のイテレーション in 開発 Dec 27, 2022
@peno022 peno022 moved this from 今のイテレーション to レビュー中 in 開発 Jan 5, 2023
@peno022
Copy link
Contributor

peno022 commented Jan 12, 2023

本番で、プラクティス一覧画面(ログインが必要なページ)でトップページにリダイレクトされて「ログインしてください」のメッセージが出ること、お問い合わせ(ログインが不要なページ)にはログインなしでアクセスできることを確認しました。クローズします。

@peno022 peno022 closed this as completed Jan 12, 2023
開発 automation moved this from レビュー中 to 完成 Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
開発
  
完成
Development

No branches or pull requests

2 participants