Yioop Repo - Seekquarry

« Back Issue #221

2013-05-07 10:48 Reported by perepamallika
2013-11-01 10:10 Marked fixed

Status: Marked fixed

Priority: Medium

Manage Groups Feature

perepamallika 2013-05-07 10:48

Hi Professor,

This is the patch that I have generated for Manage Groups feature in Yioop.

Thanks, Mallika

Additional information: This is the patch that I have generated for Manage Groups feature in Yioop.

administrator 2013-05-08 12:37

Hey Mallika,

Have you got a version since the changes I recommended yesterday.

Make sure to add me as someone monitoring your issues next time, otherwise, I don't you've changed something.

Best, Chris

perepamallika 2013-05-15 19:36

Hi Professor,

I have got a new patch for the Manage Groups feature.

-Mallika

administrator 2013-05-16 12:17

Hey Mallika,

The May 15 patch was better. Applying the patch though gives lots of whitespace messages. Then after it was applied I don't get the Manage Group option showing up at all. Did you write a function in lib/upgrade_functions.php to automatically update the database? Please take a look at the code there. Also, it is okay to give yourself author credits on files you create. When you have made a new patch, let me know.

Best, Chris

administrator 2013-11-01 10:10

Fixed by 49ce52a, thanks Mallika!

perepamallika 2013-11-01 10:10

Attachments:

ManageGroups.patch

ManageGroupsRevised.patch

YioopManageGroups.patch

ManageGroups22.patch