From 955b42689bbfae25c69e4d4d7e47f5efdd49132d Mon Sep 17 00:00:00 2001 From: Stefan Schake Date: Fri, 6 Oct 2017 02:20:23 +0200 Subject: [PATCH] Remove superfluous series_type parameter Strava API docs indicate this parameter is only relevant when also sending the resolution parameter In any case, parameter need to be sent as POST data, not in the URL This was mangling the temp series name. With this commit, Strava requests will also include the temp series --- src/Cloud/Strava.cpp | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/src/Cloud/Strava.cpp b/src/Cloud/Strava.cpp index 9b06e7c5b..80d54c918 100644 --- a/src/Cloud/Strava.cpp +++ b/src/Cloud/Strava.cpp @@ -437,15 +437,7 @@ Strava::addSamples(RideFile* ret, QString remoteid) QString urlstr = QString("https://www.strava.com/api/v3/activities/%1/streams/%2") .arg(remoteid).arg(streamsList); -#if QT_VERSION > 0x050000 - QUrlQuery params; -#else - QUrl params; -#endif - - params.addQueryItem("series_type", "time"); - - QUrl url = QUrl( urlstr + params.toString() ); + QUrl url = QUrl( urlstr ); printd("url:%s\n", url.url().toStdString().c_str()); // request using the bearer token