Tim Abbott
Tim Abbott
Merged, thanks @shubham-padia and @sahil839!
Please fix the typos in the function name -- these should all match the URLs in the API documentation. Check out the [Zulip commit message guidelines](https://zulip.readthedocs.io/en/latest/contributing/commit-discipline.html#commit-messages) for more details.
@pronebird can you edit your report to show the server version involved? https://zulip.com/help/view-zulip-version
Also is this the desktop app, or the web app? And what OS? A possibility is that your OS is making a sound when a "visual desktop notification" is triggered....
tsearch-extras is no longer required; we support vanilla postres, with dictionary files (so https://zulip.readthedocs.io/en/latest/production/postgresql.html#cloud-provider-managed-postgresql-e-g-amazon-rds applies).
@mikkeschiren do you have a recommendation for how to resolve this?
Hmm, it does look like upstream Electron fixed this and that fixed it for some other Electron apps. Unclear why that fix hasn't applied to the Zulip app.
https://github.com/electron/electron/pull/15441 and https://github.com/irccloud/irccloud-desktop/issues/86 were the things that suggested to me this was fixed upstream.
This looks good, merged, but see the note at https://github.com/zulip/zulip/pull/32046#discussion_r1826048872 for a possible code cleanup that'd be a nice follow-up PR. Thanks for the contribution @opmkumar!
Closing in favor of https://github.com/zulip/zulip/pull/33441, which continues this. Thanks for the effort @Mukul1235!