New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: functionality to assign users by user group #26044
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all four review comments are done. |
Linter is still failing as code isn't formatted. https://github.com/frappe/erpnext/wiki/Pull-Request-Checklist#linting |
Linter is not failing, i did install that utility |
You can view the linter failure logs here: https://github.com/frappe/frappe/actions/runs/8811181962/job/24184735736?pr=26044 If you installed the For now you can run |
Thanks @akhilnarang for all the help. I think we are good to go now. |
@ankush , When time permits please review and merge this PR, thanks. |
Hey, this is a good-to-have feature. However, it feels weird to have another field for this rather than merging it with the "Assign To" field. Given how mentions are handled in comments - groups only distinguished by the group icon, I would expect the same here too. |
@gavindsouza , can it be considered for future enhancement? As i have a cloud customer waiting for this feature. thanks |
@ankush , i have a cloud customer waiting for this feature. Could you please merge it. thanks |
* feat: functionality to assign users by user group * fix: incorporate four comments from review by ankush * fix: linting issues (cherry picked from commit fdc2bd8)
# [15.27.0](v15.26.0...v15.27.0) (2024-05-14) ### Bug Fixes * allow navigating to `Table MultiSelect` fields using tab ([496ae05](496ae05)) * cache github release data per bench (backport [#26382](#26382)) ([#26387](#26387)) ([6e8ef85](6e8ef85)) * **document_follow:** `following` was treated as a string not a boolean ([e0ecfc6](e0ecfc6)) * Increase folder link field size ([#26381](#26381)) ([#26384](#26384)) ([52c335c](52c335c)) * **integrations:** add back `response.raise_for_status()` ([13182e4](13182e4)) * make rename_doc work pre_model_sync ([#26419](#26419)) ([2db51ea](2db51ea)) * pass user and shared params when checking for cache keys ([#26402](#26402)) ([#26404](#26404)) ([a0eb7c1](a0eb7c1)) ### Features * functionality to assign users by user group ([#26044](#26044)) ([#26432](#26432)) ([c308931](c308931)) ### Performance Improvements * Don't update session in cache after every request ([#26308](#26308)) ([#26362](#26362)) ([c0ba12c](c0ba12c))
a new functionality to assign users using user group
below is the animated gif showing it in action