Review of "Phi: Pi-hole Indicator" version 4

Details Page Preview

Quickly access your beloved Pi-hole from GNOME Shell! Phi provides the following features: - Monitor single or multiple Pi-hole instances, - Enable/disable blocking, - Display status and main statistics, - Show version information of Pi-hole, and notify when there is an update. You can configure Phi to monitor Pi-hole only on a specific network. This prevents sending credentials to other networks and avoids unnecessary network activity on work, school, or public Wi-Fi. NEW in v2.7: GNOME 50 support, bug fixes and UX improvements. See changelog for more details.

Extension Homepage
https://github.com/ziyagenc/phi

No comments.

Diff Against

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
2.7 (28) Unreviewed
2.6 (27) Active
2.6 (26) Rejected
2.5 (25) Active
2.3 (24) Active
2.3 (23) Active
2.3 (22) Active
2.2 (21) Active
2.2 (20) Rejected
2.1 (19) Active
2.0 (18) Rejected
1.8 (17) Rejected
1.8 (16) Rejected
1.8 (15) Active
1.7 (14) Active
1.6 (13) Active
1.5 (12) Active
1.4 (11) Inactive
1.3 (10) Active
1.3 (9) Rejected
8 Inactive
7 Inactive
6 Active
5 Inactive
4 Rejected
3 Inactive
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
`lookupByUUID` shouldn't be used when provided by the entry point. Do this instead: ```js this._indicator = new PiholeIndicator(this); ``` then: ```js class PiholeIndicator extends GObject.Object { constructor(me) { ``` That's the easy fix but I suggest to create gsettings only once in the entry point to avoid creating multiple instances in different modules. to do that, you can do this instead: ```js this._indicator = new PiholeIndicator(this, this.getSettings()); ```
Ziya Genç posted a review
Thanks, updated accordingly & resubmitted.