Review of "ClipFlow Pro" version 1.2.13 (7)

Details Page Preview

A modern, powerful clipboard manager for GNOME Shell with intelligent organization, beautiful UI, and comprehensive history management.

Extension Homepage
https://github.com/nickotmazgin/clipflow-pro

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

Version Status
1.3.7 45.47 (27) Rejected
1.3.7 (26) Rejected
1.3.6 45.47 (25) Rejected
1.3.6 (24) Rejected
1.3.5 45.47 (23) Rejected
1.3.5 43.44 (22) Rejected
1.3.4 45.47 (21) Rejected
1.3.4 43.44 (20) Rejected
1.3.3 (19) Active
1.3.3 43.44 (18) Rejected
1.3.1 43.44 (17) Rejected
1.3.1 (16) Inactive
1.3.0 (15) Inactive
1.3.0 43.44 (14) Rejected
1.3.0 (13) Rejected
1.3.0 (12) Rejected
1.2.17 (11) Inactive
1.2.16 (10) Inactive
1.2.15 (9) Rejected
1.2.14 (8) Rejected
1.2.13 (7) Inactive
1.2.11 (6) Rejected
1.2.10 (5) Rejected
1.2.9 (4) Rejected
1.2.8 (3) Rejected
1.2.6 (2) Rejected
1.2.3 (1) Rejected

Previous Reviews on this Version

Ghost posted a review
Hi JustPerfection, Thank you for the feedback. I've addressed all the issues: #5 - Prefs.js debug code (lines 1244-1246): - Removed Diagnostics section from prefs.js (Enable Debug Logging switch and self-check button) - Removed _runSelfCheck() method - Removed enable-debug-logs key from schema #6 - Unnecessary try/catch wrappers: - Removed try/catch blocks that only logged errors (e.g., _destroyIndicator(), enable(), disable()) - Kept minimal try/catch only for I/O operations (file ops, JSON parsing) to prevent crashes on corrupted data - Errors now bubble with automatic backtraces ByteArray deprecation: - Verified ByteArray is completely removed (fixed in 1.2.12) - All decoding uses TextDecoder/GLib.Bytes toArray() Additional cleanup: - Removed all _debugLog() calls and method (104+ instances) - Removed _logEnabled property and enable-debug-logs setting connection - All debug/debugging code eliminated Version: 1.2.13 (version 19) Ready for review. Thanks, Nick
JustPerfection active
Hi Nick, Approve this one but as mentioned before, please remove line 1202-1204 `prefs.js`. The`buildPrefsWidget()` function isn't needed for 45+ packages. 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)