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?

One more way that we can do it is keeping the id as NAME1 automatically if the NAME is a valid aggregate group name but not a valid id.


 * 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