https://github.com/discourse/discourse/pull/32145 in core
changes the mentions a bit and introduces inline-flex,
which means that part of the content is not visible according
to capybara. Checking for `:all` fixes this.
* FEATURE: add "assigned:<name>" filter (in the /filter page)
The "assigned:<name>" filter was already available in the advanced
search but it wasn't in the /filter page.
This commit adds that filter allowing anyone to filter topics assigned
to either a specific user (using their username) or to a specific group
(using its name).
* DEV: use argument instead of instance variable
* SPEC: add spec for when the user can't assign
* WIP
* more messing around
* undos, notes to self
* WIP notes
* completed logic and added tests for existing code and implemented code
* fixed failing tests
* fixed failing tests
* added suggested changes
* added suggested changes
* fix merge
* fix linting issues
* possible flaky test
* removed comment
* further user name display changes
* implemented suggested change
* fixed bug in small action post when assigning or unassigning
* fixed breaking tests
* refactored test
---------
Co-authored-by: Juan David Martinez <juan@discourse.org>
In https://github.com/discourse/discourse/pull/30242, the topic footer buttons mobile dropdown can be replaced by the button if there's only one option.
This fixes the spec that was clicking on the dropdown button before checking there was no assign option.
* FIX: Not send notifications when it should never notify assignment
- Added silenced_assignments table and migration to store silenced assignments
- Added tests for silenced assignments
* DEV: Add annotation to model
* DEV: add empty line after early return
* DEV: lint file
Dependent on https://github.com/discourse/discourse/pull/30132 getting
merged.
The UI for the "hidden" topic footer buttons on mobile changes from
SelectKit to DMenu, so the specs needed to be updated as such.
Before this commit, the reminder was leaking the total assigned topics count in the title.
In the post raw it was ensured that it was not being leaked but the title was still leaking the count.
Also add specs and updated existing specs to ensure that the count is not leaked in the title.
* FEATURE: Add `should_notify` option to `Assigner#assign`
This option let's you control whether the added user within a group assignment should be notified or not.
* DEV: stree assign_controller file
* Update app/controllers/discourse_assign/assign_controller.rb
Co-authored-by: David Taylor <david@taylorhq.com>
---------
Co-authored-by: David Taylor <david@taylorhq.com>