Review of "Gold Price Monitor" version 29

Details Page Preview

A Gnome extension helps you tracking gold price in realtime. Service provider: goldprice.org

Extension Homepage
https://github.com/wotmshuaisi/goldpricemonitor

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

Previous Reviews on this Version

JustPerfection rejected
1. Please remove 48-50 from shell version. 2. Also null out in disable: ```js this._settings = null; ``` [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 3. `lookupByUUID()` is a bad practice (line 37 `extension.js`). You can send `this` from the entry point to the class needing it (dependency injection). 4. Add to the description that this extension is using `goldprice.org` services. 5. Change `_onDestroy()` (line 157 `extension.js`) to `destroy()`. Also change line 160 to `super.destroy()`. 6. Remove version from import (line 23 `extension.js`). Not needed. 7. Please don't store any instance of objects in a property in the default class you are exporting (line 11 `prefs.js`) that won't allow the garbage collector to do its job after window close: [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) Make it local to `fillPreferencesWindow()` function or you can clean up on: ```js window.connect('close-request', () => { // clean up here }); ``` 8. Don't use spawn command for opening the prefs window (line 62 `extension.js`). You have `this.openPreferences()` in the entry point: [Port Guide 45: openPreferences](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#extensionutils)