Damian Krzeminski wrote:
http://track.sipfoundry.org/browse/XX-5552
http://track.sipfoundry.org/browse/XX-5536


I inherited those 2 and I am sure why they are connected.

XX-5552 should be about letting users to modify their phonebooks - so I
would expect some changes to phonebook page - but it does not seem to make
any changes there.

XX-5536 about letting users to modify their contact information
(specifically avatar): which should mean changes to Contact Information tab
for the user and for the admin. The attached UI proposal actually contains
the changes to "My Information" tab which would be OK as well.
Yes, you are right - probably is my fault, because I posted the patch into a wrong place. Actually the last patch (http://track.sipfoundry.org/secure/attachment/21628/0001-XX-5552-Rebased-Laurentiu-s-patch-for-GWT-contact_in.patch),
is about 'Contact Information page' for user portal (page wrote in GWT).
That is the reason that XX-5536 is based on XX-5552 (Cosmin put the avatar in this page).
But it looks like XX-5536 relies on XX-5552, which seems to be adding new
Contact Information tab. I even tried to take 5552 just to see how it's
working but all I can see is an empty new tab.
I rebased and applied the last patch and everything worked fine.
Anybody has idea on how to disentangle it? I would happily take some
smaller patches with REST functionality and maybe admin portal support for
avatar if it's going to get us closer to the goal.
D.
Now Cosmin and I are working to integrate correctly the gravatar part (rest service for gravatar will be included in the rest service for contact information). The original patch doesn't contain any support for admin portal (only for User Portal).
Should the administrator change the avatar for registered users?

Thanks,
Laurentiu
_______________________________________________
sipx-dev mailing list [email protected]
List Archive: http://list.sipfoundry.org/archive/sipx-dev
Unsubscribe: http://list.sipfoundry.org/mailman/listinfo/sipx-dev
sipXecs IP PBX -- http://www.sipfoundry.org/

Reply via email to