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

Adapt.wait flush #2743

Closed
danielghost opened this issue May 6, 2020 · 1 comment · Fixed by #2934
Closed

Adapt.wait flush #2743

danielghost opened this issue May 6, 2020 · 1 comment · Fixed by #2934

Comments

@danielghost
Copy link
Contributor

Is the behaviour implemented via #2439 a good idea? Isn't it the responsibility of the plugin to ensure they call an Adapt.wait.end() if using an Adapt.wait.begin().

Flushing any open waits after 7 seconds means you can't intentionally block Adapt from loading, if required. For example, you may want to not call an end() and display an uncancellable notification to indicate an issue which should prevent a user from continuing. Currently the flush will end this, continue with the loading and eventually close any notifications regardless of the plugin functionality.

@olivermartinfoster
Copy link
Contributor

Yes, I agree.

danielghost added a commit to cgkineo/adapt-xAPI that referenced this issue May 7, 2020
…cellable` attribute to error notifications to determine if the notification can be closed, or to stop Adapt from loading - requires a fix for adaptlearning/adapt_framework#2743.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants