← Back to team overview

openerp-dev-web team mailing list archive

Re: [Merge] lp:~openerp-dev/openobject-client-web/widget-per-user into lp:openobject-client-web

 

Review: Needs Fixing
> It is performed as expected when server-branch is compatible.

OK, sorry, forgot about that one.

> And for demo user no record exist in `res.widget.user`, so displays all widgets including `openerp favorites` and does not display ADD and Close button.

No, that's not good, if the user has no widget then the user has no widget period end of the story.

Logic is the following, very simple:

1. widgets in res.widget.user with user_id is null are displayed for current user and don't have delete button

2. widgets in res.widget.user with user_id=uid are displayed for current user and have delete button

3. user can always add new widget to bar, even when he has none displayed

> Can't understand this comment.

The part about demo user (if no widget for user in res.widget.user, you just read everything in res.widget) is not good
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/widget-per-user/+merge/41128
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/widget-per-user.



Follow ups

References