Review of "Fedora Linux Update Indicator" version 1

Details Page Preview

GNOME Shell update indicator for Fedora Linux. See README about Gnome Console

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.

Shexli (experimental)

Shexli has not produced a result yet.

All Versions

Version Status
1 Waiting for author

Previous Reviews on this Version

JustPerfection rejected
1. Don't use `@` in schema id and path. 2. Bad schema's xml filename: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#gsettings-schemas 3. Add this to description: > This extension is not affiliated, funded, or in any way associated with Fedora brand and Red Hat Software. https://gjs.guide/extensions/review-guidelines/review-guidelines.html#copyrights-and-trademarks 4. Remove line 23 prefs.js. 5. You need to mention in the description that this extension is a fork of the other extension: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#metadata-json-must-be-well-formed 6. Don't use fedora in uuid: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#copyrights-and-trademarks 7. You cannot import `gtk` in shell process (line 26 extension.js). Use `St.IconTheme` instead: https://gjs-docs.gnome.org/st13~13-icontheme/ 8. You cannot create instances in global scope which is the same as init (line 72-76 extension.js): https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-init-for-initialization 9. Don't touch something that you won't revert on disable (line 79 extension.js). 10. Don't use same name for timeout id (line 171 and 324 extension.js). If you need any help with your extension you can ask us on: - [GNOME Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions
purejava waiting for author
Superseded by review https://extensions.gnome.org/review/46453