Commit Graph

370 Commits

Author SHA1 Message Date
David Heinemeier Hansson aef3c8ca32 No need to specify the extension now 2024-10-19 19:36:23 -07:00
David Heinemeier Hansson e3f7930dcd Unlikely we are going to need this 2024-10-19 19:35:21 -07:00
David Heinemeier Hansson b8e93c4f8a Extract render svg as a real renderer 2024-10-19 19:31:48 -07:00
David Heinemeier Hansson 15b0d6e818 Simplify join code svg rendering
There are no other formats, so dont use the multiformat structure, but
prepare for a render svg upstream.
2024-10-19 19:19:12 -07:00
David Heinemeier Hansson b53a2b8880 Fix url reference 2024-10-19 19:07:48 -07:00
David Heinemeier Hansson b7efd04a43 Use _path consistently in the controllers when not changing host 2024-10-19 19:00:04 -07:00
David Heinemeier Hansson 6c5db3826b Merge pull request #120 from basecamp/use-params-expect
Use the new params#expect syntax where applicable
2024-10-20 03:57:48 +02:00
David Heinemeier Hansson 2995c1571f Use the new params#expect syntax where applicable
To ensure we don't get spurious 500s when we can serve 400s on bad data
2024-10-19 18:53:41 -07:00
David Heinemeier Hansson 87008796b1 Merge pull request #119 from basecamp/add-caching
Add caching
2024-10-20 03:36:08 +02:00
David Heinemeier Hansson cdcf2500c1 Another spot where we need to move a Current.user check to a stimulus controller to cache 2024-10-19 18:31:21 -07:00
David Heinemeier Hansson 53afd0d93c Note that Current.user cannot be referenced within partials that must be cached 2024-10-19 17:51:09 -07:00
David Heinemeier Hansson 716e14998f Note that we need to keep all partials cacheable, so no dynamic path munging 2024-10-19 17:46:36 -07:00
David Heinemeier Hansson 1d404830f4 Must touch to bust bubble caches 2024-10-19 17:42:06 -07:00
David Heinemeier Hansson 6ee10c0ad0 Merge branch 'main' into add-caching
* main:
  Extract explaining variable and move work outside the transaction constraint
2024-10-19 17:10:41 -07:00
David Heinemeier Hansson 7bd882f33c Extract explaining variable and move work outside the transaction constraint 2024-10-19 17:10:32 -07:00
David Heinemeier Hansson 0bffcb8b86 Actually need to do all work outside of transaction loop
We can be more efficient as just as descriptive with an explaining
variable
2024-10-19 17:08:05 -07:00
David Heinemeier Hansson 942eb9fd64 Extract explaining method
And keep parent method at a consistent level of abstraction
2024-10-19 17:03:37 -07:00
David Heinemeier Hansson 32d1792cb2 Fetch bubbles index with multiget cache 2024-10-19 16:50:19 -07:00
David Heinemeier Hansson cd14fda395 Cache bubble partial on itself 2024-10-19 16:50:05 -07:00
David Heinemeier Hansson 5d06e5449f Etag the bubbles#show 2024-10-19 16:49:51 -07:00
David Heinemeier Hansson 23d81c07c1 Use Rails 8.0 rc1 2024-10-19 16:42:13 -07:00
David Heinemeier Hansson e2c9048581 Only load from app, not all plugins etc
This will prevent loading trix for example
2024-10-18 12:49:12 -07:00
Jose Farias b349e31d88 Reorder routes 2024-10-18 11:46:50 -06:00
Jose Farias fdfd7a963a Merge pull request #116 from basecamp/bubble-filter
Pull out Bucket::BubbleFilter
2024-10-18 11:17:34 -06:00
Jose Farias d9d473723b Clean up Bucket::BubbleFilter initializer 2024-10-18 11:15:53 -06:00
Jose Farias 09c003431e Pull out Bucket::BubbleFilter 2024-10-18 11:11:23 -06:00
Jose Farias 224cd14fe6 Scope view lookup to Current.user 2024-10-17 23:17:27 -06:00
Jose Farias bd2f4fc476 Fix tag_filters and assignee_filters calls 2024-10-17 23:02:59 -06:00
Jose Farias 1d9d120db6 Merge pull request #115 from basecamp/move-filtered-bubbles-to-model
Move bubble filtering to the model
2024-10-17 22:55:24 -06:00
Jose Farias e7cc426fdc Move bubble filtering to the model 2024-10-17 22:51:02 -06:00
Jose Farias 91401c2e75 Put latest buckets first 2024-10-17 22:29:15 -06:00
Jose Farias e38a7fadda Merge pull request #113 from basecamp/persisted-filters
Persisted filters
2024-10-17 22:17:43 -06:00
Jose Farias 54cea4eb75 clear_assignees_if_unassigned -> clear_assignees 2024-10-17 22:09:34 -06:00
Jose Farias be43dbc693 Remove unnecessary compact_blank 2024-10-17 22:03:07 -06:00
Jose Farias c570fa7e4c Appease zeitwerk 2024-10-17 22:01:25 -06:00
Jose Farias 783f0ec83a Revert "Appease zeitwerk"
This reverts commit 87ebabe260.
2024-10-17 21:53:42 -06:00
Jose Farias 87ebabe260 Appease zeitwerk 2024-10-17 21:44:49 -06:00
Jose Farias c0df006ebc Consolidate bucket_view_helper and filters_helper 2024-10-17 21:37:48 -06:00
Jose Farias c9e6ecda58 bucket_view_form_tag -> bubble_filter_form_tag 2024-10-17 21:35:21 -06:00
Jose Farias 6dc01479b7 Fix search form 2024-10-17 21:34:48 -06:00
Jose Farias 9c2b101624 Memoize view_filter_params 2024-10-17 20:58:11 -06:00
Jose Farias 3542298394 @view might be nil in BubblesController#index 2024-10-17 20:56:52 -06:00
Jose Farias 862c854377 querying_unassigned_status -> unassigned_filter_activated 2024-10-17 20:55:11 -06:00
Jose Farias 0c27ede5ea Clean up #set_view 2024-10-17 20:53:07 -06:00
Jose Farias 919092a1f8 Disambiguate view_filter_params and buble_filter_params 2024-10-17 20:50:45 -06:00
David Heinemeier Hansson 8504e911ab No longer used 2024-10-17 17:59:01 -07:00
Jose Farias a3cbf4d9fc Fix that assignee and tag filters wouldn't overwrite themselves 2024-10-17 18:00:05 -06:00
Jose Farias 28eca68421 Fix that assignee and tag filters would overwrite each other 2024-10-17 17:52:24 -06:00
Jose Farias a7a9c46327 Put tag summary first 2024-10-17 17:46:49 -06:00
Jose Farias 6b54ad6a72 bucket_view -> view 2024-10-17 17:43:30 -06:00