Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(qchat): use mp3 instead of ogg #198

Merged
merged 1 commit into from
Oct 4, 2024
Merged

Conversation

gounux
Copy link
Contributor

@gounux gounux commented Oct 3, 2024

No description provided.

Copy link

sonarcloud bot commented Oct 3, 2024

@gounux gounux marked this pull request as ready for review October 3, 2024 16:48
@gounux
Copy link
Contributor Author

gounux commented Oct 3, 2024

testé sur win 10 QGIS 3.34.9, ça marche inch'allah

@gounux gounux requested a review from Guts October 3, 2024 16:49
@Guts
Copy link
Member

Guts commented Oct 4, 2024

Pour info, je viens de tester sur Windows 11, QGIS 3.34.9 et les sons sont bien joués quand on les sélectionne dans les settings. Je me suis envoyé un message depuis https://gischat.geotribu.fr/ où je me notifie et j'ai bien la notif QgsMessageBar et le son.

Je pensais que le son n'était joué que lorsqu'on est notifié mais c'est pour chacun des messages envoyés ?

@Guts Guts added the bug Houston, we've got a problem... label Oct 4, 2024
@gounux
Copy link
Contributor Author

gounux commented Oct 4, 2024

Je pensais que le son n'était joué que lorsqu'on est notifié mais c'est pour chacun des messages envoyés ?

Oui effectivement, ça risque de faire beaucoup de sons si y'a du monde ...
Tu penses que ça se change ?

@Guts
Copy link
Member

Guts commented Oct 4, 2024

Yep, dans une autre PR.

@gounux gounux merged commit 2052276 into main Oct 4, 2024
10 checks passed
@gounux gounux deleted the fix/fix-sounds-on-windows branch October 4, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Houston, we've got a problem...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants