Remove defer usage in `devtools/client/webconsole/webconsole-ui.js`
Categories
(DevTools :: Console, enhancement, P3)
Tracking
(Not tracked)
People
(Reporter: nchevobbe, Unassigned, Mentored)
References
(Blocks 1 open bug)
Details
Attachments
(1 obsolete file)
devtools/client/webconsole/webconsole-ui.js#11 uses defer in destroy and _initConnection.
These 2 usages could be moved to use the standard new Promise instead.
|
Reporter | |
Comment 1•5 years ago
|
||
Paarmita, would you want to work on this bug?
|
|
Reporter | |
Updated•5 years ago
|
|
||
Comment 3•5 years ago
|
||
Checked the changes with build and linting
|
||
Comment 4•5 years ago
|
||
Hi, I would like to work on this. Should I proceed?
|
|
||
Updated•5 years ago
|
|
|
Reporter | |
Comment 5•5 years ago
|
||
Hello Khyati.
The bug is currently assigned to Paarmita.
Paarmita, are you still working on this/plan to work on this?
|
|
||
Comment 6•5 years ago
|
||
(In reply to Nicolas Chevobbe from comment #5)
Hello Khyati.
The bug is currently assigned to Paarmita.Paarmita, are you still working on this/plan to work on this?
Since she has been inactive from so many days, can I proceed?
|
|
Reporter | |
Comment 7•5 years ago
|
||
it's not uncommon for people to only have a few day per month to contribute, but that doesn't mean they gave up on the bug :)
So let's wait until Paarmita answers.
I'll ping you another good first bug if you want
|
|
Reporter | |
Comment 8•5 years ago
|
||
clearing assignee as we don't have news for 2 months now
Khyati are you still interested in fixing this bug?
|
|
||
Comment 9•5 years ago
|
||
(In reply to Nicolas Chevobbe [:nchevobbe] from comment #8)
clearing assignee as we don't have news for 2 months now
Khyati are you still interested in fixing this bug?
Yes, ofcourse.
|
|
Reporter | |
Comment 10•5 years ago
|
||
(In reply to Khyati Agarwal from comment #9)
Yes, ofcourse.
Great! The bug is now yours. Let me know if you need anything.
|
||
Comment 11•5 years ago
|
||
Hi Khyati. Are you still interested in fixing this bug? If not, no problem at all, just let us know so we can make this bug available to others.
|
|
||
Updated•5 years ago
|
|
||
Comment 12•5 years ago
|
||
I'll handle this.
|
|
Reporter | |
Comment 13•5 years ago
|
||
(In reply to r2hkri from comment #12)
I'll handle this.
Sorry, this probably will be fixed by a patch I'm working on :/
|
|
||
Comment 14•5 years ago
|
||
(In reply to Nicolas Chevobbe [:nchevobbe] from comment #13)
(In reply to r2hkri from comment #12)
I'll handle this.
Sorry, this probably will be fixed by a patch I'm working on :/
No problem, thanks for letting me know.
|
||
Updated•5 years ago
|
|
|
Reporter | |
Comment 15•5 years ago
|
||
there's no more defer usage in this file
Description
•