Review of "NordVPN Quick Toggle" version 5

Details Page Preview

GNOME extension that shows a quick toggle to connect/disconnect NordVPN. This extension is not affiliated, funded, or in any way associated with NordVPN.

Extension Homepage
https://github.com/Wedaxi/NordVPN-Quick-Toggle

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Version Status
12 Active
11 Inactive
10 Inactive
9 Inactive
8 Inactive
7 Inactive
6 Inactive
5 Inactive
4 Rejected
3 Inactive
2 Inactive
1 Rejected

Previous Reviews on this Version

JustPerfection active
Thanks! To avoid double timeout on multiple calls please add this after line 11 `asyncHandler.js`: ```js if (this._timeoutId) { clearTimeout(this._timeoutId); } ``` or call `this.clear()` in the first line of the promise callback.
wedaxi posted a review
Thank you! I will add that next time I push an update, for now since asyncHandler is only used in one place there won't be double timeout because I do the clear in line 162 in extension.js