Skip to content
This repository has been archived by the owner on Oct 9, 2019. It is now read-only.

Promise.done is deprecated. #45

Open
iplusplus42 opened this issue Jan 19, 2017 · 3 comments · May be fixed by #50
Open

Promise.done is deprecated. #45

iplusplus42 opened this issue Jan 19, 2017 · 3 comments · May be fixed by #50

Comments

@iplusplus42
Copy link

Atom now uses ES6 Promises instead of Q. Call promise.then instead of promise.done

Promise.done (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:1214:5)
Object.addPreviewForEditor (/Users/Andy/.atom/packages/rst-preview/lib/rst-preview.coffee:140:10)
Object.toggle (/Users/Andy/.atom/packages/rst-preview/lib/rst-preview.coffee:116:21)
atom-workspace.atom.commands.add.rst-preview:toggle (/Users/Andy/.atom/packages/rst-preview/lib/rst-preview.coffee:54:26)
CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:259:29)
<unknown> (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:59)
@gvarisco gvarisco linked a pull request Sep 1, 2017 that will close this issue
@Fenrihr
Copy link

Fenrihr commented Apr 2, 2019

up ++

@leszekuchacz
Copy link

up +++

@raghavkaranth
Copy link

up ++

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

Successfully merging a pull request may close this issue.

4 participants