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

[MERGE] mail, *: several UX/UI improvements #29191

Closed
wants to merge 11 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@alexkuhn
Contributor

alexkuhn commented Nov 30, 2018

*: bus, im_livechat, mail_bot, website_livechat

Several improvements for Discuss app:

  1. Improved "is typing..." notification
  2. Merged private & public channels
  3. Quick channel search
  4. No push notification right after accepting them, and transparent odoo bot icon in native notification
  5. Selected emojis are inserted in composer based on cursor position
  6. Set surname to people for DMs name
  7. Discard replying to a message in Inbox
  8. Broadcast "seen" notification ('v' and 'vv' on messages)

[1] image1, image2
[2] image1, image2
[3] image1, image2
[4] image
[5] video
[6] image1, image2, image3
[7] image
[8] image1, image2

Task-ID 1881001

@robodoo robodoo added the seen 🙂 label Nov 30, 2018

@alexkuhn alexkuhn self-assigned this Nov 30, 2018

@robodoo robodoo added the CI 🤖 label Nov 30, 2018

@alexkuhn alexkuhn force-pushed the odoo-dev:master-mail-aku branch from 2565825 to d5ef458 Dec 4, 2018

@robodoo robodoo removed the CI 🤖 label Dec 4, 2018

@alexkuhn alexkuhn force-pushed the odoo-dev:master-mail-aku branch 2 times, most recently from c19c4c0 to cd9d996 Dec 4, 2018

@robodoo robodoo added the CI 🤖 label Dec 4, 2018

@alexkuhn alexkuhn force-pushed the odoo-dev:master-mail-aku branch from cd9d996 to 1884246 Dec 4, 2018

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Dec 4, 2018

@alexkuhn alexkuhn force-pushed the odoo-dev:master-mail-aku branch from 1884246 to 03a9770 Dec 5, 2018

@robodoo robodoo removed the CI 🤖 label Dec 5, 2018

@alexkuhn alexkuhn force-pushed the odoo-dev:master-mail-aku branch 2 times, most recently from 8391984 to 5d1ee30 Dec 5, 2018

@robodoo robodoo added the CI 🤖 label Dec 5, 2018

Show resolved Hide resolved addons/mail/models/mail_channel.py Outdated
Show resolved Hide resolved addons/mail/models/mail_channel.py Outdated
Show resolved Hide resolved addons/mail/models/mail_channel.py Outdated
Show resolved Hide resolved addons/mail/models/mail_channel.py Outdated
Show resolved Hide resolved addons/mail/models/mail_channel.py Outdated
Show resolved Hide resolved addons/mail/models/mail_channel.py Outdated
Show resolved Hide resolved addons/mail/models/mail_channel.py Outdated

@alexkuhn alexkuhn force-pushed the odoo-dev:master-mail-aku branch from 5d1ee30 to 60bd34a Dec 6, 2018

@robodoo robodoo removed the CI 🤖 label Dec 6, 2018

@alexkuhn alexkuhn force-pushed the odoo-dev:master-mail-aku branch 3 times, most recently from e5dbdd4 to ccf711e Dec 6, 2018

@robodoo robodoo added the CI 🤖 label Dec 6, 2018

@alexkuhn alexkuhn force-pushed the odoo-dev:master-mail-aku branch from ccf711e to f7dffd7 Dec 6, 2018

@robodoo robodoo removed the CI 🤖 label Dec 6, 2018

@alexkuhn alexkuhn force-pushed the odoo-dev:master-mail-aku branch 2 times, most recently from 38c1f79 to 1e863ee Dec 6, 2018

@robodoo robodoo added the CI 🤖 label Dec 6, 2018

@Xavier-Do

This comment has been minimized.

Contributor

Xavier-Do commented Dec 6, 2018

robodoo rebase-merge

robodoo added a commit that referenced this pull request Dec 6, 2018

[MERGE] mail, *: several UX/UI improvements
*: bus, im_livechat, mail_bot, website_livechat

Several improvements for Discuss app:

1. Improved "is typing..." notification
2. Merged private & public channels
3. Quick channel search
4. No push notification right after accepting them, and transparent odoo bot icon in native notification
5. Selected emojis are inserted in composer based on cursor position
6. Set surname to people for DMs name
7. Discard replying to a message in Inbox
8. Broadcast "seen" notification ('v' and 'vv' on messages)

[1] [image1](https://drive.google.com/file/d/11_H6ZJh0wFaegAV0fTLdc68z37nhVKnW/view?usp=sharing), [image2](https://drive.google.com/file/d/1s9g_hUCMesuTLAqpeGMwLWzHCYnzanr6/view?usp=sharing)
[2] [image1](https://drive.google.com/file/d/1WUBqc2lScfWV8Yrtq6Zk4MO4jC0CdaiG/view?usp=sharing), [image2](https://drive.google.com/file/d/13XVcW0Nookwea_4xz02mIwPi5dPhXu18/view?usp=sharing)
[3] [image1](https://drive.google.com/file/d/1FuV3xH9cK6qR7ljd5vELOHc4icvHIJTF/view?usp=sharing), [image2](https://drive.google.com/file/d/1hwuu5ejlj9ou02NF5XmL-XHTiD-Ty6XT/view?usp=sharing)
[4] [image](https://drive.google.com/file/d/1si1TYtCpdyOgw9IRaMWmxuUb1d8M4ccF/view?usp=sharing)
[5] [video](https://drive.google.com/file/d/19knQ47F1wzlNvxT2nJH3A9V34wO1D9Bn/view?usp=sharing)
[6] [image1](https://drive.google.com/file/d/11ot_DtySN2cIN_bM8n7fhFV2JXNoz0lH/view?usp=sharing), [image2](https://drive.google.com/file/d/1aNyapzKePIRg903woCo3kXJhy-ZFwfEG/view?usp=sharing), [image3](https://drive.google.com/file/d/1AtVlOG67YjsS8ax1ErWhXfSDOtKhLQhC/view?usp=sharing)
[7] [image](https://drive.google.com/file/d/1U-VEut7M7-t9WzdKVagfyy6ZrbOJrflp/view?usp=sharing)
[8] [image1](https://drive.google.com/file/d/1_32q8iqPBhW4sPhAtY5mIO968gj1o9o-/view?usp=sharing), [image2](https://drive.google.com/file/d/1Ne6HMEgJxHAWYN3e3SlrF6Qqg70LDvAW/view?usp=sharing)

Task-ID 1881001

closes #29191
@robodoo

This comment has been minimized.

Contributor

robodoo commented Dec 6, 2018

Staging failed: ci/runbot on 5e9e7e66a2c7f21fa3b2dd67eee246c4347cb402 (view more at http://runbot.odoo.com/runbot/build/411192)

@robodoo robodoo added the error 🙅 label Dec 6, 2018

@alexkuhn

This comment has been minimized.

Contributor

alexkuhn commented Dec 6, 2018

robodoo retry

@robodoo robodoo added CI 🤖 r+ 👌 and removed error 🙅 labels Dec 6, 2018

robodoo added a commit that referenced this pull request Dec 6, 2018

[MERGE] mail, *: several UX/UI improvements
*: bus, im_livechat, mail_bot, website_livechat

Several improvements for Discuss app:

1. Improved "is typing..." notification
2. Merged private & public channels
3. Quick channel search
4. No push notification right after accepting them, and transparent odoo bot icon in native notification
5. Selected emojis are inserted in composer based on cursor position
6. Set surname to people for DMs name
7. Discard replying to a message in Inbox
8. Broadcast "seen" notification ('v' and 'vv' on messages)

[1] [image1](https://drive.google.com/file/d/11_H6ZJh0wFaegAV0fTLdc68z37nhVKnW/view?usp=sharing), [image2](https://drive.google.com/file/d/1s9g_hUCMesuTLAqpeGMwLWzHCYnzanr6/view?usp=sharing)
[2] [image1](https://drive.google.com/file/d/1WUBqc2lScfWV8Yrtq6Zk4MO4jC0CdaiG/view?usp=sharing), [image2](https://drive.google.com/file/d/13XVcW0Nookwea_4xz02mIwPi5dPhXu18/view?usp=sharing)
[3] [image1](https://drive.google.com/file/d/1FuV3xH9cK6qR7ljd5vELOHc4icvHIJTF/view?usp=sharing), [image2](https://drive.google.com/file/d/1hwuu5ejlj9ou02NF5XmL-XHTiD-Ty6XT/view?usp=sharing)
[4] [image](https://drive.google.com/file/d/1si1TYtCpdyOgw9IRaMWmxuUb1d8M4ccF/view?usp=sharing)
[5] [video](https://drive.google.com/file/d/19knQ47F1wzlNvxT2nJH3A9V34wO1D9Bn/view?usp=sharing)
[6] [image1](https://drive.google.com/file/d/11ot_DtySN2cIN_bM8n7fhFV2JXNoz0lH/view?usp=sharing), [image2](https://drive.google.com/file/d/1aNyapzKePIRg903woCo3kXJhy-ZFwfEG/view?usp=sharing), [image3](https://drive.google.com/file/d/1AtVlOG67YjsS8ax1ErWhXfSDOtKhLQhC/view?usp=sharing)
[7] [image](https://drive.google.com/file/d/1U-VEut7M7-t9WzdKVagfyy6ZrbOJrflp/view?usp=sharing)
[8] [image1](https://drive.google.com/file/d/1_32q8iqPBhW4sPhAtY5mIO968gj1o9o-/view?usp=sharing), [image2](https://drive.google.com/file/d/1Ne6HMEgJxHAWYN3e3SlrF6Qqg70LDvAW/view?usp=sharing)

Task-ID 1881001

closes #29191

@robodoo robodoo added the merging 👷 label Dec 6, 2018

robodoo added a commit that referenced this pull request Dec 7, 2018

[MERGE] mail, *: several UX/UI improvements
*: bus, im_livechat, mail_bot, website_livechat

Several improvements for Discuss app:

1. Improved "is typing..." notification
2. Merged private & public channels
3. Quick channel search
4. No push notification right after accepting them, and transparent odoo bot icon in native notification
5. Selected emojis are inserted in composer based on cursor position
6. Set surname to people for DMs name
7. Discard replying to a message in Inbox
8. Broadcast "seen" notification ('v' and 'vv' on messages)

[1] [image1](https://drive.google.com/file/d/11_H6ZJh0wFaegAV0fTLdc68z37nhVKnW/view?usp=sharing), [image2](https://drive.google.com/file/d/1s9g_hUCMesuTLAqpeGMwLWzHCYnzanr6/view?usp=sharing)
[2] [image1](https://drive.google.com/file/d/1WUBqc2lScfWV8Yrtq6Zk4MO4jC0CdaiG/view?usp=sharing), [image2](https://drive.google.com/file/d/13XVcW0Nookwea_4xz02mIwPi5dPhXu18/view?usp=sharing)
[3] [image1](https://drive.google.com/file/d/1FuV3xH9cK6qR7ljd5vELOHc4icvHIJTF/view?usp=sharing), [image2](https://drive.google.com/file/d/1hwuu5ejlj9ou02NF5XmL-XHTiD-Ty6XT/view?usp=sharing)
[4] [image](https://drive.google.com/file/d/1si1TYtCpdyOgw9IRaMWmxuUb1d8M4ccF/view?usp=sharing)
[5] [video](https://drive.google.com/file/d/19knQ47F1wzlNvxT2nJH3A9V34wO1D9Bn/view?usp=sharing)
[6] [image1](https://drive.google.com/file/d/11ot_DtySN2cIN_bM8n7fhFV2JXNoz0lH/view?usp=sharing), [image2](https://drive.google.com/file/d/1aNyapzKePIRg903woCo3kXJhy-ZFwfEG/view?usp=sharing), [image3](https://drive.google.com/file/d/1AtVlOG67YjsS8ax1ErWhXfSDOtKhLQhC/view?usp=sharing)
[7] [image](https://drive.google.com/file/d/1U-VEut7M7-t9WzdKVagfyy6ZrbOJrflp/view?usp=sharing)
[8] [image1](https://drive.google.com/file/d/1_32q8iqPBhW4sPhAtY5mIO968gj1o9o-/view?usp=sharing), [image2](https://drive.google.com/file/d/1Ne6HMEgJxHAWYN3e3SlrF6Qqg70LDvAW/view?usp=sharing)

Task-ID 1881001

closes #29191
@robodoo

This comment has been minimized.

Contributor

robodoo commented Dec 7, 2018

Staging failed: ci/runbot (view more at http://runbot.odoo.com/runbot/build/411256)

@alexkuhn

This comment has been minimized.

Contributor

alexkuhn commented Dec 7, 2018

robodoo retry

@robodoo robodoo added CI 🤖 r+ 👌 and removed error 🙅 labels Dec 7, 2018

robodoo added a commit that referenced this pull request Dec 7, 2018

[MERGE] mail, *: several UX/UI improvements
*: bus, im_livechat, mail_bot, website_livechat

Several improvements for Discuss app:

1. Improved "is typing..." notification
2. Merged private & public channels
3. Quick channel search
4. No push notification right after accepting them, and transparent odoo bot icon in native notification
5. Selected emojis are inserted in composer based on cursor position
6. Set surname to people for DMs name
7. Discard replying to a message in Inbox
8. Broadcast "seen" notification ('v' and 'vv' on messages)

[1] [image1](https://drive.google.com/file/d/11_H6ZJh0wFaegAV0fTLdc68z37nhVKnW/view?usp=sharing), [image2](https://drive.google.com/file/d/1s9g_hUCMesuTLAqpeGMwLWzHCYnzanr6/view?usp=sharing)
[2] [image1](https://drive.google.com/file/d/1WUBqc2lScfWV8Yrtq6Zk4MO4jC0CdaiG/view?usp=sharing), [image2](https://drive.google.com/file/d/13XVcW0Nookwea_4xz02mIwPi5dPhXu18/view?usp=sharing)
[3] [image1](https://drive.google.com/file/d/1FuV3xH9cK6qR7ljd5vELOHc4icvHIJTF/view?usp=sharing), [image2](https://drive.google.com/file/d/1hwuu5ejlj9ou02NF5XmL-XHTiD-Ty6XT/view?usp=sharing)
[4] [image](https://drive.google.com/file/d/1si1TYtCpdyOgw9IRaMWmxuUb1d8M4ccF/view?usp=sharing)
[5] [video](https://drive.google.com/file/d/19knQ47F1wzlNvxT2nJH3A9V34wO1D9Bn/view?usp=sharing)
[6] [image1](https://drive.google.com/file/d/11ot_DtySN2cIN_bM8n7fhFV2JXNoz0lH/view?usp=sharing), [image2](https://drive.google.com/file/d/1aNyapzKePIRg903woCo3kXJhy-ZFwfEG/view?usp=sharing), [image3](https://drive.google.com/file/d/1AtVlOG67YjsS8ax1ErWhXfSDOtKhLQhC/view?usp=sharing)
[7] [image](https://drive.google.com/file/d/1U-VEut7M7-t9WzdKVagfyy6ZrbOJrflp/view?usp=sharing)
[8] [image1](https://drive.google.com/file/d/1_32q8iqPBhW4sPhAtY5mIO968gj1o9o-/view?usp=sharing), [image2](https://drive.google.com/file/d/1Ne6HMEgJxHAWYN3e3SlrF6Qqg70LDvAW/view?usp=sharing)

Task-ID 1881001

closes #29191
@robodoo

This comment has been minimized.

Contributor

robodoo commented Dec 7, 2018

Merged, thanks!

@robodoo robodoo closed this Dec 7, 2018

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