Commit Graph

1589 Commits

Author SHA1 Message Date
Jorge Manrubia 2ec2043e7b Merge pull request #783 from basecamp/activity-summaries
AI summaries
2025-07-23 12:27:25 +02:00
Jorge Manrubia fc1ff7bf5e Fix tests 2025-07-23 12:21:20 +02:00
Jorge Manrubia dcb0e5077d Move model options to helper 2025-07-23 11:57:22 +02:00
Jorge Manrubia c0b385b19b Tidy up the admin tool for prompts 2025-07-23 11:55:44 +02:00
Jorge Manrubia c3ae05833e Make models selectable 2025-07-23 11:02:34 +02:00
Jason Zimdars ae14912cbe Add an animated effect for the generating state 2025-07-22 22:33:19 -05:00
Jason Zimdars 7280077531 Remove debug 2025-07-22 18:59:02 -05:00
Jason Zimdars be29567718 Wrap them both in the container so the generating state looks right 2025-07-22 18:15:59 -05:00
Jason Zimdars ab315cdd75 Style activity summaries 2025-07-22 18:12:05 -05:00
Andy Smith 9489e23fc4 Add margins to the input instead of a wrapper. 2025-07-22 14:32:43 -05:00
Andy Smith 424b33979c Removed redundant div 2025-07-22 14:29:05 -05:00
Jorge Manrubia 7e5bdf59ec Prefer sandbox over debug for the resource name 2025-07-22 15:57:12 +02:00
Jorge Manrubia b803e659df Remove tab 2025-07-22 15:28:23 +02:00
Jorge Manrubia e3d9001861 Move condition outside to avoid double nesting 2025-07-22 15:13:39 +02:00
Jorge Manrubia 8f6e81a687 Only summarize today if there are at least 10 events
https://3.basecamp.com/2914079/buckets/37331921/todos/8877635813#__recording_8879434338
2025-07-22 15:04:55 +02:00
Jorge Manrubia 6ba1884fb0 Fix: summaries where failing to generate with filters 2025-07-22 14:59:38 +02:00
Jorge Manrubia 6d26195bc7 Add a debug prompt endpoint so that we can play with the prompts live 2025-07-22 14:33:34 +02:00
Jorge Manrubia 9c6f1354c4 Broadcast and show summaries on the fly 2025-07-22 12:26:32 +02:00
Jorge Manrubia 7cd459f2b8 AI Summaries: basic abstractions, functional version 2025-07-22 11:57:39 +02:00
Andy Smith ed0492e2f4 Use smaller action size on mobile 2025-07-21 16:30:56 -05:00
Andy Smith 67df61b57d Dry up utility classes for popup__group 2025-07-21 13:59:33 -05:00
Andy Smith 7e0df47c8f Same padding for all popup__items 2025-07-21 13:38:08 -05:00
Andy Smith 37cc6a6955 popup__items full-width by default 2025-07-21 13:37:33 -05:00
Andy Smith d3db752f39 Remove redundant flex-grow class from popup__items 2025-07-21 13:35:24 -05:00
Andy Smith 1e62ae8c94 Set min-width on popup__items 2025-07-21 13:32:56 -05:00
Mike Dalessio 8e13ce5b78 Drop Current.account and just use Account.sole
I had reintroduced this in dacb53b8 after David removed it in
6bbf68a4, but now I agree we don't need it.

Also, fix a CORS error when trying to redirect a user to an untenanted
path after login (see
https://fizzy.37signals.com/5986089/collections/2/cards/1032).
2025-07-20 13:00:09 -04:00
Mike Dalessio 692e563d3f Call blob.preview for previewable attachments
undoing a mistake introduced in ef6198d3
2025-07-19 09:08:59 -04:00
Mike Dalessio ef6198d394 Make sure the image variants we use are preprocessed
to avoid implicitly creating a variant during a GET to a read replica,
which will raise an exception and cause the image to fail to load.

This feels pretty brittle, to be honest, and we'll need to be careful
in the future about creating and using new variants; we will probably
need to backfill the new variant for existing images because we can't
guarantee that they will be created implicitly/lazily if the GET lands
on the replica.

ref: https://fizzy.37signals.com/5986089/collections/7/cards/942
2025-07-18 15:43:31 -04:00
Jason Zimdars 468a29c9ad Give them screen reader labels
See: https://fizzy.37signals.com/5986089/collections/2/cards/1106
2025-07-16 11:33:33 -05:00
Jason Zimdars f0235e7e07 Not needed
See: https://fizzy.37signals.com/5986089/collections/2/cards/1108
2025-07-16 11:08:06 -05:00
Jason Zimdars ec8084074f Add missing lang attribute
See: https://fizzy.37signals.com/5986089/collections/2/cards/1110
2025-07-16 11:05:23 -05:00
Jason Zimdars 15a809855d We're only concerned with truncating the placeholder, scope to that 2025-07-16 08:30:03 -05:00
Andy Smith 8d60592ccd Merge pull request #720 from basecamp/notification-links
Notification links
2025-07-10 17:04:31 -05:00
Jason Zimdars 70c2cc62a1 Merge branch 'main' into public-collection-cards
* main:
  Revert "Adjust borders"
  Don't allow wrapping initially
  Adjust borders
  Truncate long commands
  Try using env safe insets to avoid overlapping elements on mobile
  Correct asset width and remove caption input focus ring
  Revert "Make a focus-ring global variable"
  Make a focus-ring global variable
  Set `id` explicitly so the `label for` reference works
  Higher resolution image
  When adding a card out of context, default to most recently accessed collection
  Fine tuning to handle different attachment types
  Replace helper with view lookup
  Make nil safe
  Extract attachments to a partial
  Remove `card_published` check and render its own template
  Extract common concern to examine attachments
  This is a hack but we should be showing the card description and attachments for new cards
  Display attachments in events for comments
  Indicate cards with attachments in the description
2025-07-10 16:33:51 -05:00
Jason Zimdars 74ac6fae9e Merge branch 'mobile-insets'
* mobile-insets:
  Revert "Adjust borders"
  Don't allow wrapping initially
  Adjust borders
  Try using env safe insets to avoid overlapping elements on mobile
2025-07-10 16:31:34 -05:00
Jason Zimdars 74e0cd3af8 Don't allow wrapping initially 2025-07-10 14:50:36 -05:00
Andy Smith 0585c5f1e1 Truncate long commands 2025-07-10 14:44:22 -05:00
Jason Zimdars 0d1c0f4e75 Try using env safe insets to avoid overlapping elements on mobile 2025-07-10 14:26:58 -05:00
Jason Zimdars 4d65d2be85 Set id explicitly so the label for reference works 2025-07-10 12:56:59 -05:00
Andy Smith 5162677c44 Align dialog, shrink hat, canvas background 2025-07-10 11:20:37 -05:00
Jorge Manrubia 81560c73dd Use existing direct route 2025-07-10 14:57:20 +02:00
Jason Zimdars ed37bf0ff8 When adding a card out of context, default to most recently accessed collection 2025-07-09 15:37:39 -05:00
Jason Zimdars 32e967c1f9 Fine tuning to handle different attachment types 2025-07-09 14:46:33 -05:00
Jason Zimdars 7a058a18d9 Merge branch 'main' into card-attachments
* main:
  Add circle-on-mobile button class
2025-07-09 11:22:41 -05:00
Jason Zimdars a60eedf186 Merge branch 'main' into public-collection-cards
* main:
  Add circle-on-mobile button class
2025-07-09 11:20:41 -05:00
Jason Zimdars cd022fcc1c Not fit for public 2025-07-09 11:10:41 -05:00
Jason Zimdars 55f92e73f7 Not needed here 2025-07-09 11:08:09 -05:00
Jason Zimdars b309f85d13 Match text style 2025-07-09 11:03:43 -05:00
Jason Zimdars 90419084c3 Ensure step checkboxes reflect status and can't be toggled 2025-07-09 11:01:54 -05:00
Jason Zimdars b142b08c05 Include lightbox viewer for background images 2025-07-09 10:58:23 -05:00