5
11
Fork
You've already forked tomato
1

Always authenticate_user in controllers #39

Merged
edwardloveall merged 1 commit from el-auth-by-default into main 2025年10月28日 15:46:17 +01:00

Before, we were calling this in every controller individually; opt-in. For safety, let's always call it make make controller opt-out if they don't need it (like for the root page).

Before, we were calling this in every controller individually; opt-in. For safety, let's always call it make make controller opt-out if they don't need it (like for the root page).
thetizzo left a comment
Copy link

This PR made me relax a little 😂

This PR made me relax a little 😂
edwardloveall force-pushed el-auth-by-default from 6dee43d835
Some checks failed
Setup Successful
Jest Successful
Static Analysis Successful
RSpec Failed
to 89ca54e1da
Some checks failed
Setup Successful
Jest Successful
Static Analysis Successful
RSpec Failed
2025年10月28日 15:04:34 +01:00 Compare
Author
Owner
Copy link

Tests are flaky, but passed locally. Merging

Tests are flaky, but passed locally. Merging
edwardloveall deleted branch el-auth-by-default 2025年10月28日 15:46:17 +01:00
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rootable/tomato!39
Reference in a new issue
rootable/tomato
No description provided.
Delete branch "el-auth-by-default"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?