Commit Graph

6311 Commits

Author SHA1 Message Date
Jorge Manrubia bfc0b75e6c Not used 2025-11-03 07:16:11 +01:00
David Heinemeier Hansson 9caf39ad2c Use modern strong params syntax 2025-11-02 20:26:12 +01:00
David Heinemeier Hansson e4cdc15b6a Fix tests 2025-11-02 20:25:11 +01:00
David Heinemeier Hansson 39dc2b6ab9 Prefer explicit turbo stream templates 2025-11-02 20:22:15 +01:00
David Heinemeier Hansson cf503bfba3 Prefer templates for turbo streams in all but the very simplest cases
(And probably even then)
2025-11-02 20:20:01 +01:00
David Heinemeier Hansson 3ccc93294f Style 2025-11-02 18:13:23 +01:00
David Heinemeier Hansson 3ff7a88294 Style 2025-11-02 18:11:00 +01:00
David Heinemeier Hansson fd61b4426c Conventional to do includes first 2025-11-02 18:09:30 +01:00
David Heinemeier Hansson 828ac5d2c1 Conventional order of actions 2025-11-02 18:08:49 +01:00
David Heinemeier Hansson 43069b1ea4 Extract Users::JoinsController from overloaded UsersController
Good code smell is when the before_action callbacks stack up but can't
be shared across actions
2025-11-02 17:49:25 +01:00
David Heinemeier Hansson ada599f6a2 Use latest Rails 2025-11-02 17:31:35 +01:00
David Heinemeier Hansson bced7405df Extract events loading from overloaded UsersController 2025-11-02 17:31:28 +01:00
David Heinemeier Hansson 6a62df470c No longer used 2025-11-02 16:34:29 +01:00
David Heinemeier Hansson 877e62dec5 No CR when there are only two statements 2025-11-02 16:30:49 +01:00
David Heinemeier Hansson 76c87f6b67 Test updating account settings 2025-11-02 16:04:51 +01:00
David Heinemeier Hansson 02bd57089d Use before_action to pull up account assignment 2025-11-02 16:04:44 +01:00
David Heinemeier Hansson c834cce504 Simplify join code creation and validation and test controller
cc @monorkin
2025-11-02 15:58:33 +01:00
David Heinemeier Hansson f09183bc9a Don't want to balloon txn, but also don't want to end up with a state with broken data 2025-11-02 15:46:05 +01:00
David Heinemeier Hansson 2c9845eb62 No need to hide that the top-level entropy is tied to the Account 2025-11-02 15:24:55 +01:00
David Heinemeier Hansson a1543f2660 Inline anemic helper method with no additional explaining purpose 2025-11-02 14:20:07 +01:00
David Heinemeier Hansson d6c88736bd No need for validation when we have rely on update!/create! on db constraints 2025-11-02 14:19:26 +01:00
David Heinemeier Hansson 0af74bfa11 Style 2025-11-02 14:17:46 +01:00
David Heinemeier Hansson 966aa51a37 Get rid of unnecessary touch jobs
touch_all on 10K cards took 6ms in testing
2025-11-02 14:15:53 +01:00
David Heinemeier Hansson 105585c26c Merge pull request #1451 from basecamp/entropies
Entropy::Configuration -> Entropy
2025-11-02 13:59:43 +01:00
David Heinemeier Hansson b1c8dc1e64 Need even more grace to ensure success 2025-11-02 13:56:31 +01:00
David Heinemeier Hansson 7052d6ecec Fix style 2025-11-02 13:54:46 +01:00
David Heinemeier Hansson 2c4159d11a Style 2025-11-02 13:29:07 +01:00
David Heinemeier Hansson fb395c8b07 Keep all entropy definitions together in the concern 2025-11-02 13:17:55 +01:00
David Heinemeier Hansson b3474ec97d Rename Entropy::Configuration to just Entropy 2025-11-02 13:11:41 +01:00
David Heinemeier Hansson d29e4dcdcf No CR when there are only two statements in total 2025-11-02 12:49:54 +01:00
David Heinemeier Hansson 72d0008df0 Only restrict resource actions when there's a specific story to tell 2025-11-02 12:49:17 +01:00
David Heinemeier Hansson e6459cede3 Use existing authorization method 2025-11-02 08:34:06 +01:00
David Heinemeier Hansson 6ceb4ad870 Merge pull request #1441 from basecamp/events-daily-cut-off
Ensure there is a limit of 100 events per column per day
2025-11-02 08:24:34 +01:00
David Heinemeier Hansson d6deb3ac5f Setup signoff too 2025-11-02 08:22:32 +01:00
David Heinemeier Hansson 3bba72bf19 Wait a little to ensure attachments have loaded 2025-11-02 08:19:22 +01:00
David Heinemeier Hansson 8e48167774 Merge branch 'main' into events-daily-cut-off 2025-11-02 08:12:02 +01:00
Jason Zimdars b07345724a Merge pull request #1438 from basecamp/profile-layout
Put profile boxes side-by-side
2025-10-31 17:11:30 -05:00
Jason Zimdars 56d30d6d97 Merge pull request #1439 from basecamp/event-title-style
Underline event titles
2025-10-31 17:09:04 -05:00
Jason Zimdars 693fe6bf50 Merge pull request #1435 from basecamp/bar-icon-align
Align bar shortcuts better
2025-10-31 17:07:49 -05:00
Jason Zimdars 2a8d361c5a Merge pull request #1440 from basecamp/new-board-in-nav
Shift Add Board around in the nav
2025-10-31 17:05:48 -05:00
Jason Zimdars b56ea485c0 Merge pull request #1442 from basecamp/new-card-hotkey
Add hotkey to create a new card
2025-10-31 17:04:02 -05:00
Jason Zimdars 62de1b4b55 Merge pull request #1445 from basecamp/animate-popups
Animate popups
2025-10-31 17:02:22 -05:00
Jason Zimdars 4b7cd8828f Style notice 2025-10-31 16:58:21 -05:00
Jason Zimdars 46f692de0d Merge pull request #1447 from basecamp/closing-soon
Need a special human name for this one, move to a method
2025-10-31 16:43:38 -05:00
Jason Zimdars ccbc7cd416 Need a special human name for this one, move to a method 2025-10-31 16:41:50 -05:00
Jason Zimdars 8d58190483 Merge pull request #1446 from basecamp/tweak-brand-line
Polish branding line
2025-10-31 16:31:34 -05:00
Jason Zimdars f851cf8ff3 Polish branding line 2025-10-31 16:29:48 -05:00
Jason Zimdars 9533de4e72 Merge pull request #1444 from basecamp/card-back-button
Treat the collection link like a back button rather than a title
2025-10-31 16:25:39 -05:00
Jason Zimdars 54c4f09f0f + Newline EOF 2025-10-31 16:24:32 -05:00
Andy Smith 2cab272ee8 Remove unused class 2025-10-31 16:23:22 -05:00