diff options
| author | Xavier Morel <xmo@odoo.com> | 2020-09-21 11:17:35 +0000 |
|---|---|---|
| committer | Xavier Morel <xmo@odoo.com> | 2020-09-21 11:17:35 +0000 |
| commit | d252b0c6311fd389295e980e6f9725673ce8b650 (patch) | |
| tree | 997a97c444ffbe4af5815608a4332a740170d0b5 /addons/mail/static/src/models/thread/thread.js | |
| parent | 64fc3c98d26d2a41180560744aac6f35fa77497a (diff) | |
[FIX] auth_totp: tour not working on a base install
This is a followup to #57826: turns out discuss isn't necessarily
installed (who knew?), so the tour would break if mail was not
explicitly installed alongside base, which is not great.
So instead of opening discuss to try and fix the profile race
condition, reload the *entire web client* by navigating to `/web`.
Further discovery: async run functions don't actually do anything in
tours, not sure where I got that, maybe used one for the convenience
of `await` then the next time around having decided to wipe any
knowledge of tours I had, I figured it wasn't possible that tours were
so stupid as to not support promises.
So yeah, can't just return a "pending" promise in order to force the
tour manager to wait until the page has reloaded to carry on, that
doesn't work. Instead use marker classes. Also remove the `async run`
I had in the rest of the file to avoid being misleading, use a marker
class for the one step where it probably matters.
closes odoo/odoo#58119
Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
Diffstat (limited to 'addons/mail/static/src/models/thread/thread.js')
0 files changed, 0 insertions, 0 deletions
