Review of "Gotify Notifications" version 6

Details Page Preview

Displays persistent Gotify notifications in GNOME. Independent of GNOME's native notification system

Extension Homepage
https://github.com/dodog/gotify-notifications

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) warning 1

Shexli found 1 issue that may need reviewer attention.

EGO-A-004 warning

extension files should not contain excessive ungated console logging

File contains 31 ungated console.log/warn/error calls (threshold: 5).

No excessive logging

  • extension.js:322
    console.error('Gotify: Soup request failed:', errorMsg)
  • extension.js:708
    console.error('Gotify: Failed to poll:', error)
  • extension.js:33
    console.log(`Gotify: Creating custom notification: ${title}`)
  • extension.js:77
    console.log('Gotify: Closing notification')
  • extension.js:168
    console.log('Gotify: Synchronously clearing all notifications')
  • extension.js:196
    console.log('Gotify: All notifications cleared synchronously')
  • extension.js:204
    console.log('Gotify: Clearing all notifications with animation')
  • extension.js:215
    console.log('Gotify: Removing notification')
  • extension.js:284
    console.log('Gotify: Making HTTP request with Soup to:', url)
  • extension.js:311
    console.log('Gotify: HTTP status:', status)

All Versions

Version Status
6 Active
5 Inactive
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

fmuellner active