Review of "Fedora Linux Update Indicator" version 1

Details Page Preview

GNOME Shell update indicator for Fedora Linux. This is based on Arch Linux Updates Indicator by Raphaƫl Rochet. This extension is not affiliated, funded, or in any way associated with Fedora brand and Red Hat Software.

Extension Homepage
https://github.com/purejava/fedora-update

No comments.

FAQ

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 move line 32 prefs.js to `fillPreferencesWindow`. You can attach it to the window to avoid garbage collecting before window close. 2. Do that on enable (line 74 extension.js). and delete that on disable (not nulling that out). 3. It's better to move line 581-585 to `FedoraUpdateIndicator`. 4. You can add `gettext-domain` to metadata.json to make it easier for 45 port: https://gjs.guide/extensions/overview/anatomy.html#metadata-json-required 5. You should clean up `UPDATES_LIST` in disable: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects