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
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
c7b3b45977
Merge pull request #391 from basecamp/cards-pagination-2
...
Paginate collection cards
2025-04-10 15:10:54 +02:00
695247ae9b
Paginate collection cards
2025-04-10 15:09:56 +02:00
aa7694ce7b
Update fixtures and test
2025-04-09 17:28:24 -05:00
723e6d94f5
Rename bubbles => cards
2025-04-09 14:50:58 +02:00
9bbb59a166
Test for Bubbles::RecoversController
2025-04-07 14:30:05 +01:00
23892296da
Move readings under bubbles and out of double nesting
2025-04-07 14:06:56 +02:00
481be72162
Move bubbles publish out from the double nesting
2025-04-07 13:55:12 +02:00
ce0c6146be
Move bubble pops out from double nesting
2025-04-07 13:45:51 +02:00
e4f7837ebe
Remove double nesting under bubbles for engagement
2025-04-07 13:30:31 +02:00
c06802856e
Move boosts under bubbles
...
But also remove the need for a bucket nesting
2025-04-06 21:04:28 +02:00
4fd1472fae
Move bubble watch out from the needless bucket nesting
2025-04-06 19:22:40 +02:00
eb0614c44f
Use a more ergonomic API for watching
...
Passing a boolean vs using explicit on/off methods feel too low level
and doesnt read right in most of these cases.
2025-04-06 19:17:26 +02:00
a58fcd3123
Nicer API
2025-04-06 13:26:10 +02:00
cdd78b41ee
Tag exclusively by title
2025-04-06 13:19:22 +02:00
540316e639
Move taggings under bubbles
2025-04-05 21:54:48 +02:00
25b89ce8a9
Trailing CR
2025-04-05 17:18:18 +02:00
5134985418
Simplify the assignments flow
...
We now submit on every selection, so we dont need a whole form and
checkboxes to do it.
This needs some design tune-up after the change, though. cc @jzimdars
2025-04-05 17:14:57 +02:00
d0953d6f87
Pull assignments out of the double nesting
2025-04-05 15:59:51 +02:00
b4c0fec7a6
Move Assignments under Bubble
...
Since that what we are assigning
2025-04-05 15:51:53 +02:00
e69c1bd0a1
Break pins out of the double nesting
...
Its cumbersome and needless
2025-04-05 15:45:16 +02:00
866a80e133
Fix up the tests
2025-04-05 15:26:49 +02:00
bfb7ed1216
Move pins to my space
...
We used the my/ space in HEY to reflect anything that was primarily
hinged off Current.user. Good to use here too.
2025-04-05 14:57:24 +02:00
7494dad91e
Drop the toggles scope for taggings as well
2025-04-05 14:20:06 +02:00
384de24132
Get rid of odd toggles concept
...
This is just assignments.
2025-04-05 14:16:04 +02:00
f960b3fc83
Add tests
2025-04-04 13:35:54 +02:00
dbae700554
Merge pull request #346 from basecamp/two-columns-refactor
...
Refactor collections perma + support for doing/considering
2025-04-04 08:29:17 +02:00
05fc73e2ae
Merge pull request #334 from basecamp/user-admins
...
Add admin role and animate admin toggle
2025-04-04 08:29:03 +02:00
1ea70d7d3b
Merge branch 'main' into two-columns-refactor
...
* main:
Fix image icons
Revert "Merge pull request #345 from basecamp/cascade-layers"
Fine tune notifications states
Remove empty helper
Use an icon for the button
Show button on collection page and notif settings
Add involvement types to accesses
Extract a couple components
Add cascade layers to all CSS files
2025-04-03 19:30:37 -05:00
cfe91c25c9
Add admin role, make sure only admins can remove and promote other users
2025-04-03 17:43:29 +02:00
7cb2a476c4
Fix tests
2025-04-03 17:41:55 +02:00
0820badcc3
Add involvement types to accesses
...
This provides a way to set the level of involvement that a user has with
a collection, and from which we determine the level of notifications to
send. Users can be access-only, watching, or being notified about
everything.
If you're access-only, you won't get an notifications. If you're
watching, you'll only get notifications for the items you're watching
(which includes the items you've been assigned, have commented on, etc).
If you're set to everything you'll get notifications about all activity
in that collection.
This change replaces our previous concept of subscriptions. Where
previously you'd subscribe to a collection to get notifications in it,
now you'll simply set the notification level on your access.
2025-04-03 16:35:13 +01:00
ac0baed663
Add some missing tests
2025-04-03 14:46:35 +02:00
b7c2369988
Fix setting the workflow on a collection
...
The previous implementation would delete all accesses if the bucket
was not all_access.
I think it's simpler to move the bucket workflow into a separate
Buckets::WorkflowsController, rather than deal with different sets of
params from different forms in the existing BucketsController.
2025-03-30 09:24:56 -04:00
David Heinemeier Hansson