08d8b2e5ff
Revert "Replace House with Lexical"
2025-05-29 14:22:27 +02:00
34eb8f1672
Format
2025-05-29 12:38:07 +02:00
ecf82d8703
Make tests pass with action text
2025-05-29 12:37:16 +02:00
10b2aad647
Replace House with lexical-powered actiontext
2025-05-23 11:41:08 +02:00
ea4a3e458d
Update test
2025-05-19 19:08:39 -05:00
e0e17d2bd0
Add test for the commands controller
2025-05-16 16:12:09 +02:00
b539cd1ed1
Fix tests
2025-05-13 16:39:11 -05:00
5c6a53a1bc
WIP new sorting
2025-05-13 12:34:07 -05:00
b017a3fd67
Destroy unconfirmed commands so that they don't end up in the user history
2025-05-07 14:43:11 +02:00
8697cabbe2
Show confirmation when submitting certain commands
...
https://3.basecamp.com/2914079/buckets/37331921/todos/8617430347
2025-05-07 10:20:32 +02:00
22c3c33c0b
Merge pull request #460 from basecamp/fizzy-do-initial
...
Fizzy Do (initial)
2025-05-07 08:57:37 +02:00
26614c5f02
Fix test
2025-05-07 08:56:32 +02:00
2d22bef203
We can't rely on count now so test that the specific collection no longer exists
2025-05-06 12:27:02 -05:00
37bf992d67
Add controller tests for undo
2025-05-06 15:17:57 +02:00
e8adf15ba7
Add controller tests
2025-05-06 15:07:02 +02:00
2b98db9e66
Flatten out comments and messages
...
https://37s.fizzy.37signals.com/collections/693169850/cards/999008989#comment_994777841
2025-04-29 19:21:18 +02:00
e90b062440
Configure the autoclose period at the collection level
2025-04-25 13:29:16 +02:00
b08326ccbe
Merge branch 'main' into plain-text-mentions
...
* main: (48 commits)
Improve layout and truncation
This is annoying
Fix summary
Remove index columns
Same treatment for the events index
New menu filters and navigates
Don't test for _Added by_ events
Indention
@collection is always available when card scoped
Style
Remove check for _Added by..._ line
The placeholder attribute is on the `house-md` element
Duh, no before destroy commit, but this is still not right!
No long transactions!
Don't display published event in threads
But feels more essentially part of the User, so list first
Not related to the other two
Missing controller
Style
Errant `bubbles` => `cards` replacement
...
# Conflicts:
# app/models/card.rb
# app/models/user.rb
# db/schema.rb
# db/schema_cache.yml
2025-04-23 17:24:01 +02:00
3d28203224
Use singular staging resource and beef up testing
2025-04-22 14:44:55 +02:00
8aa99e34b4
Persist notification's creator
...
Instead of delegating. With a polymorphic relationship, relying on a certain
attribute implies doing things like aliasing "mentioner" to "creator" or similar.
2025-04-22 13:48:43 +02:00
b0aae3ad9e
Reapply "Add a description field for cards"
...
This reverts commit a4cc143b09
2025-04-21 16:45:24 +02:00
a4cc143b09
Revert "Add a description field for cards"
2025-04-21 16:41:33 +02:00
9e921b85c0
Merge branch 'main' into add-card-description
2025-04-19 20:00:43 +02:00
3003105ac9
Flesh out the rest of the controller tests
2025-04-18 16:16:04 +02:00
f6cd21f7cb
Extract default stages into something that will be created together with a new Workflow in the model
2025-04-18 16:07:01 +02:00
fd21838f88
Add assert_in_body helper
...
Coming upstream
2025-04-18 16:06:30 +02:00
44a5215da7
Merge branch 'main' into add-card-description
...
* main:
Fix failing test (we're no longer redirecting here
Display the empty message via CSS when there are no unread notifications
Hide Previously Seen section when empty with CSS
2025-04-17 17:08:49 -05:00
c61fa77d6b
Fix failing test (we're no longer redirecting here
...
See: #411
2025-04-17 17:03:18 -05:00
04f9a0f83a
Merge branch 'main' into add-card-description
...
* main: (33 commits)
Run beamer accessories in production only
Less noisy logging in production
Inline perma to container partial
Use card replacement for card updates too
Extract helper method to keep partial at a uniform level of abstraction
Remove duplicated conditional
Style
Note that we dont want to keep reloading these frames
I like this even better
Suffix qualifier is unnnecessary
Group partials for container
Match domain language
Fix trailing space
Use card rerendering instead of redirects
Better name
Stick with domain language
Fix tests
Rerender instead of redirect
Style
Extract partials that can be rerendered
...
2025-04-17 16:40:23 -05:00
7adbd8d634
Use card rerendering instead of redirects
2025-04-17 15:20:27 +02:00
06da295f44
Fix tests
2025-04-17 14:56:53 +02:00
7b5a1c0232
Fix test
2025-04-17 14:32:26 +02:00
fc0cf3917f
Pin without redirect
...
Use turbo stream response to save on the redirect latency
2025-04-17 14:05:26 +02:00
a5de4863fa
Allow for multiple notifications to be read
...
cc @kevinmcconnell
2025-04-17 11:39:37 +02:00
47d0ae98a1
Remove DraftCommenting
2025-04-16 19:01:21 -05:00
91194d1b66
Fixed up controller test
2025-04-16 16:37:06 +02:00
50e4587dd9
Fix trailing spaces
2025-04-16 11:56:21 +02:00
375afeac7b
Fix test
2025-04-16 11:55:35 +02:00
73f272f132
Use one controller for all notification readings
2025-04-15 19:14:50 +02:00
93178636cd
Flesh out all the tests
2025-04-15 15:41:01 +02:00
2e8df45680
Flesh out tests for users controller
2025-04-15 15:36:11 +02:00
dbf61f4fc3
Use _path in all cases where we are not potentially changing the domain
2025-04-13 08:17:36 +02:00
a106b1b6a7
Join the two users controllers but split out role setting
2025-04-12 20:30:47 +02:00
6bbf68a4f9
The tenanted db is the account scope
2025-04-12 20:06:34 +02:00
760cbb6c99
Drop boosts
...
Killed by design.
2025-04-12 18:52:54 +02:00
7861b7c087
Just open one turbo stream per card
...
Per comment is excessive and needless
2025-04-11 17:58:11 +02:00
79700162fc
Use correct stream name
...
comment, :comments makes no sense for a stream for the reactions
2025-04-11 17:47:35 +02:00
f4ce57e308
Move comments under cards
2025-04-11 16:21:44 +02:00
4f3f322528
Rename inconsistency: go with goldness
2025-04-11 10:22:18 +02:00
1bac0b84b3
Add tests
2025-04-11 09:36:24 +02:00
Jorge Manrubia