User:Kunalgrover05/AggregateGroups

Changes for Special:AggregateGroups
These are the fixes that we are aiming at for Special:AggregateGroups
 * Allow aggregate group name and group description to be changed on Special:AggregateGroups. - [Bug 36298]
 * Make Special:AggregateGroups output a special page when permissions insufficient (bug number?)

Approach/Implementation

 * Have an edit button beside the remove button to change the name and description.
 * Implement different display forms for the 2 cases.

Current progress

 * Reading up and understanding the code
 * Implemented a method in API to change the name and description.
 * Implemented UI method to use API

Current challenges
Now multiple groups can have the same name as well. Is it better to create a new group followed by deleting the old one?
 * It breaks the searching groups using aggregate group id.


 * Nikerabbit suggested to have the same group id even on updation


 * This brings this bug into place, suppose a new group is created with a certain name and then the name is changed. We can never again create a group with the same name. Is it better to use the Copy group and delete existing group way?


 * The error part in mw.Api doesn't work for me.


 * Solved with Nikerabbit's help


 * Need to add the error message for invalid update, couldn't figure out where to add it.

Gerrit patches
https://gerrit.wikimedia.org/r/#/c/135872 https://gerrit.wikimedia.org/r/#/c/135874
 * API
 * Front end

Further tasks
Making Special:AggregateGroups read only if permissions not available