 David Heinemeier Hansson
|
e8a730c239
|
Fix constant
|
2025-04-13 20:36:07 +02:00 |
|
|
David Heinemeier Hansson
|
34232c0e31
|
Use record lookup
|
2025-04-13 20:30:03 +02:00 |
|
|
David Heinemeier Hansson
|
d9a9690137
|
Always use sole
|
2025-04-13 20:29:10 +02:00 |
|
|
David Heinemeier Hansson
|
9e0b9c08c2
|
Explained less and extracted nothing of value
|
2025-04-13 20:28:35 +02:00 |
|
|
David Heinemeier Hansson
|
33278469ef
|
Weird double negative
|
2025-04-13 20:26:58 +02:00 |
|
|
David Heinemeier Hansson
|
a5df54015e
|
Remove anemic and indirect method that explained nothing
|
2025-04-13 20:25:59 +02:00 |
|
|
David Heinemeier Hansson
|
e2a931b657
|
Can use record lookup for workflows
|
2025-04-13 20:24:16 +02:00 |
|
|
David Heinemeier Hansson
|
366fe06113
|
Access control has already been done through the card
|
2025-04-13 20:22:53 +02:00 |
|
|
David Heinemeier Hansson
|
e12ef119da
|
Style
|
2025-04-13 20:17:24 +02:00 |
|
|
David Heinemeier Hansson
|
dbf61f4fc3
|
Use _path in all cases where we are not potentially changing the domain
|
2025-04-13 08:17:36 +02:00 |
|
|
David Heinemeier Hansson
|
7b60748796
|
Clearer
|
2025-04-13 08:11:25 +02:00 |
|
|
David Heinemeier Hansson
|
4ef46ccd19
|
Spacing
|
2025-04-13 08:10:47 +02:00 |
|
|
David Heinemeier Hansson
|
5b524254b9
|
Merge pull request #403 from basecamp/commit-to-single-account
The tenanted db is the account scope
|
2025-04-13 07:58:12 +02:00 |
|
|
David Heinemeier Hansson
|
cf93da6886
|
Get rid of card_title
We can just set untitled on assignment of empty
|
2025-04-12 21:29:56 +02:00 |
|
|
David Heinemeier Hansson
|
a106b1b6a7
|
Join the two users controllers but split out role setting
|
2025-04-12 20:30:47 +02:00 |
|
|
David Heinemeier Hansson
|
c463f342bf
|
Ensure user is returned
|
2025-04-12 20:30:28 +02:00 |
|
|
David Heinemeier Hansson
|
a579e67057
|
We are not using the parameter
|
2025-04-12 20:08:42 +02:00 |
|
|
David Heinemeier Hansson
|
6bbf68a4f9
|
The tenanted db is the account scope
|
2025-04-12 20:06:34 +02:00 |
|
|
David Heinemeier Hansson
|
19dbd5b325
|
Merge pull request #402 from basecamp/drop-boosts
Drop boosts
|
2025-04-12 18:54:46 +02:00 |
|
|
David Heinemeier Hansson
|
760cbb6c99
|
Drop boosts
Killed by design.
|
2025-04-12 18:52:54 +02:00 |
|
|
David Heinemeier Hansson
|
1b732c41c7
|
Use render svg
|
2025-04-12 18:15:05 +02:00 |
|
|
David Heinemeier Hansson
|
120e8da1d5
|
Unused
|
2025-04-12 14:14:34 +02:00 |
|
|
David Heinemeier Hansson
|
4723b7b504
|
Style
|
2025-04-12 14:13:20 +02:00 |
|
|
David Heinemeier Hansson
|
d3c5c86b21
|
Style
|
2025-04-12 14:06:11 +02:00 |
|
|
David Heinemeier Hansson
|
a9c6f8bceb
|
No reuse likely
|
2025-04-12 13:51:47 +02:00 |
|
|
David Heinemeier Hansson
|
b36b2f04a9
|
Too clever to have the redirect_to directly on the factory method return
|
2025-04-12 13:49:23 +02:00 |
|
|
David Heinemeier Hansson
|
19c537af64
|
Inline anemic method
|
2025-04-12 13:48:03 +02:00 |
|
|
David Heinemeier Hansson
|
9b89562755
|
Merge pull request #398 from basecamp/comments-refactoring
Comments refactoring
|
2025-04-12 13:45:10 +02:00 |
|
|
David Heinemeier Hansson
|
1a70be06c6
|
Fix trailing CR
|
2025-04-12 13:37:31 +02:00 |
|
|
David Heinemeier Hansson
|
706178f3c4
|
Better naming
|
2025-04-12 13:33:54 +02:00 |
|
|
David Heinemeier Hansson
|
d4a31fec6d
|
Dont overload the main transaction
Even if we are going to get rid of this cleanup_events stuff.
|
2025-04-12 13:33:06 +02:00 |
|
|
David Heinemeier Hansson
|
2357c75063
|
Reads better with a verb first
|
2025-04-12 13:09:49 +02:00 |
|
|
David Heinemeier Hansson
|
e8a839d5f6
|
We have a callback dependency to do this actually
|
2025-04-12 11:47:08 +02:00 |
|
|
David Heinemeier Hansson
|
0031ce34ea
|
Correct order
|
2025-04-12 11:46:17 +02:00 |
|
|
David Heinemeier Hansson
|
e4efe446e6
|
Simlpe callback is actually enough
Since the Comment already knows about the card.
|
2025-04-12 11:45:34 +02:00 |
|
|
David Heinemeier Hansson
|
f2c2b0492e
|
Have to solve this in an overhaul of events
|
2025-04-12 11:42:50 +02:00 |
|
|
David Heinemeier Hansson
|
f5b30456a0
|
Comments arent part of event summaries any more
|
2025-04-12 11:42:41 +02:00 |
|
|
David Heinemeier Hansson
|
e75d2bcaa1
|
Scopes last
|
2025-04-12 11:37:39 +02:00 |
|
|
David Heinemeier Hansson
|
d8f9511e08
|
Unused
|
2025-04-12 11:34:56 +02:00 |
|
|
David Heinemeier Hansson
|
1a9a65fffb
|
Unused
|
2025-04-12 11:31:50 +02:00 |
|
|
David Heinemeier Hansson
|
2f8a9bed0e
|
There is only ever one account per database
So there is no protection in scoping things by account cc @jorgemanrubia
|
2025-04-12 11:30:28 +02:00 |
|
|
David Heinemeier Hansson
|
9359cac330
|
Style
|
2025-04-12 11:24:02 +02:00 |
|
|
David Heinemeier Hansson
|
b2b9c99745
|
Style
|
2025-04-12 11:23:48 +02:00 |
|
|
David Heinemeier Hansson
|
81a89dbda1
|
Comments should always have messages
|
2025-04-12 11:23:45 +02:00 |
|
|
David Heinemeier Hansson
|
b8c3ac8145
|
Cache busting should happen as a comment in the partial, not as an actual key component
|
2025-04-12 11:22:31 +02:00 |
|
|
David Heinemeier Hansson
|
cef8899bda
|
Comments are not currently used as notifiables
|
2025-04-12 11:18:59 +02:00 |
|
|
David Heinemeier Hansson
|
9f1c3ae046
|
Note my discontent
|
2025-04-12 11:17:01 +02:00 |
|
|
David Heinemeier Hansson
|
1c4e0d3ccc
|
Not using the destroy callback
|
2025-04-12 11:11:30 +02:00 |
|
|
David Heinemeier Hansson
|
63c05220ce
|
Clearer naming
|
2025-04-12 11:09:55 +02:00 |
|
|
David Heinemeier Hansson
|
ecffdc2af4
|
Spacing
|
2025-04-12 11:07:27 +02:00 |
|