Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Observers #25

Closed
jaumef opened this issue Feb 15, 2018 · 2 comments
Closed

Improve Observers #25

jaumef opened this issue Feb 15, 2018 · 2 comments

Comments

@jaumef
Copy link
Contributor

jaumef commented Feb 15, 2018

The current observers field is a comma-separated string:

image

It would be better to use a Many2Many with res_partner_address

Also would be great to have a BCC Observers as "secret observers"

@jaumef jaumef self-assigned this Feb 15, 2018
@jaumef jaumef added the feature label Feb 15, 2018
@ecarreras
Copy link
Member

To don't break the behavior, maybe we can convert the new field to the comma separated list

@jaumef
Copy link
Contributor Author

jaumef commented Mar 19, 2018

TODO

Replace the Watchers field with a new page with "many2many" fields alongisde with their comma separated list related field. We need both lists:

  • Watchers (CC)
  • Secret watchers (BCC)

The many2many fields are related to a res.partner.address.
Then we may do an on_change method for those "many2many" to add the related addresses to the string field without duplicated addresses and respecting the addresses that are added manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants