-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Show better error messages when talking to MusicBrainz. #19
Conversation
@@ -75,7 +75,9 @@ void AcoustidClient::requestFinished() { | |||
int id = m_requests.take(reply); | |||
|
|||
if (reply->attribute(QNetworkRequest::HttpStatusCodeAttribute).toInt() != 200) { | |||
emit finished(id, QString()); | |||
emit networkError( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, can you put parens after emit?
emit(foo);
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well I can do
emit foo(bar);
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
emit(foo(bar));
is how we do it everywhere in Mixxx. Today Qt defines emit as an empty statement which is why that works but in the future they could change that.
Looks good other than those minor comments. Thanks! |
Cool -- thanks for the fixes! |
Show better error messages when talking to MusicBrainz.
CI: Update Docker image for GitHub pre-commit workflow
2.1 beta downloads
https://bugs.launchpad.net/mixxx/+bug/1184158
Show a more meaningfull error message when mixxx fails to reload metadata from the internet.
If we get a HTTPErrorCode from the service it will show the error and service.
If we don't get a HTTPErrorCode a more generic site is shown naming the service that failed.
To test this behaviour just mess up the URL variables in acoustidclient.cpp and musicbrainzclient.cpp
I haven't touched the .ui files much. I would like to have some words centered and in bold but I can't figure out how this is done without using HTML in the QLabel. Jus do you know a way to do this in QtDesigner?