Skip to content

Commit

Permalink
Empty metrics issue fixed! (#105)
Browse files Browse the repository at this point in the history
* empty metrics issue fixed!

* - SDK Version update

Co-authored-by: Zahid Zafar <>
  • Loading branch information
ZahidZafar authored Jan 26, 2023
1 parent d32de0c commit 1828d37
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@

22.06.3
* Fixed a bug where empty metrics were sent with session begin request.

22.06.2
* Fixed a bug where metrics were not sent with session begin request.

Expand Down
2 changes: 1 addition & 1 deletion include/countly/constants.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
#include "nlohmann/json.hpp"

#define COUNTLY_SDK_NAME "cpp-native-unknown"
#define COUNTLY_SDK_VERSION "22.06.2"
#define COUNTLY_SDK_VERSION "22.06.3"
#define COUNTLY_POST_THRESHOLD 2000
#define COUNTLY_KEEPALIVE_INTERVAL 3000
#define COUNTLY_MAX_EVENTS_DEFAULT 200
Expand Down
2 changes: 1 addition & 1 deletion src/countly.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -518,7 +518,7 @@ bool Countly::beginSession() {
session_params.erase("user_details");
}

if (configuration->metrics.contains("metrics")) {
if (configuration->metrics.size() > 0) {
data["metrics"] = configuration->metrics.dump();
}

Expand Down

0 comments on commit 1828d37

Please sign in to comment.