1. Please add this to the description: > This extension is not affiliated, funded, or in any way associated with Spotify. 2. Remove `license` from `metadata.json`. 3. Don't need to use any parameters for `this.getSettings()` since you already have that in `metadata.json`. 4. You should call `abort()` on destroy (line 8 `core/LyricsClient.js`): [`Soup.Session.abort()`](https://gjs-docs.gnome.org/soup30~3.0/soup.session#method-abort) Don't forget to calll destroy on the instance. 5. Add to the description that this extension is using `lrclib.net` services. 6. Please call `this._stopAnimation()` before line 72 `ui/slider.js`. 7. You should also call `this._stopAnimation()` on `MediaSlider.destroy` (line 78 `ui/slider.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 8. Remove timeout on destroy: - line 205 `ui/popup.js` - line 149 `ui/LyricsWidget.js` [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 9. Move it to the destroy function rather than connecting signal (line 67 `ui/panel.js`). 10. Avoid sending large code. It's hard to review. You can also add `donations` to the `metadata.json`, so people can donate to you if they want: [Extension Anatomy: donations](https://gjs.guide/extensions/overview/anatomy.html#donations) If you need any help with your extension you can ask us on: - [GNOME Extensions Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions