* main: (21 commits)
squash! Remove session controller actions other than delete
Remove the invitation codes for now
Remove session controller actions other than delete
Account.signal_account is no longer optional
Remove local authentication code
Drop the password column from the users table.
Script to create Users for existing 37id accounts
Configure Mission Control to skip tenanted authentication
Script to update URLs and attachments in cards and comments
Make sure seeds are creating sluggable tenants
Script to rename tenants after the QB account id
Update the signups tests to work with untenanted paths
Update the Command classes to handle paths with account slugs
Update bin/dev to point at the new tenant URLs
Patch Active Storage controllers to generate slugged URLs
Point Action Cable at the slugged cable endpoint
Extract pieces from the tenanting initializer
Patch Turbo to use the request script name
Use the new AR::Tenanted convention for action cable connections
Use a tenant resolver that recognizes the account slug
...
I proposed this upstream in https://github.com/rails/rails/pull/55250,
but it's not clear if the presence of script_name should _always_
prefix generated URLs, so for now it's local to Fizzy.
where the closure no longer exists, possibly due to being
reopened.
Previously, fragment caching generally prevented any problems, but if
the cache is busted for any reason (like Andy did in c7eeb56c), then
the previous logic would raise an exception like "undefined method
'reason' for nil".
So, if the closure no longer exists, let's just note that it was
closed, and by who.
* main:
Clear the comments cache
Add context for search screens
Needs changed for both states
Use lighter icon and update asset
Add btn--circle class to user edit form
Organize button variants
Untangle `avatar btn` crossover styles
Remove redundant classes and styles for avatars
Adjust footer line height
Tweak padding
I went with placing htis logic at the command level since it plays well with the composite structure, where you don't know whether you are handling one command or many commands, and you want to aggregate all the errors. I didn't leverage the existing support with rails' model errors because that customizing those message is cumbersome (either i18n or using custom validation logic for everything).
- Introduces a helper method for setting the filterable value
- Updates `filter_controller.js` to use `data-filter-text-value` attribute populated with converted content. This was previous `data-value` in the earlier implementation but ignored in the current on.
- Also uses the same helper to make @mentions diacritic insensitive
Andy Smith