-
-
Notifications
You must be signed in to change notification settings - Fork 585
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
Lazy loading: fix end-to-end encryption rooms #683
Conversation
bwindels
commented
Aug 6, 2018
•
edited
Loading
edited
- don't break device key tracking
- when offline, make sure you can send messages in e2e room when back online
when loading the encrypted events from storage, the code would wait for the encryption target users, which would never come because you would only load them when viewing the room. This disabled starting to track the devices in the room when the inhibitDeviceQuery is set.
src/utils.js
Outdated
}); | ||
promise.resolve = resolve; | ||
return promise; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think bluebird are fairly keen to avoid to avoid you doing this, ie. it's very strongly geared towards using new Promise()
directly, so we should probably drink their kool-aid rather than try to work against the library.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As all our target platforms all support native promises, it would be nice to drop bluebird at some point, and just use the DOM APIs on Promise. But given the changes we discussed this morning, I don't need this anymore, so getting rid of it.
40703d6
to
0ae0770
Compare