Protei/PromucFlow_constructor
2
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Use email as id for users and fix signup events not being reported (#1067)

Browse Source
This commit is contained in:
Shrikant Sharat Kandula 2020-10-07 21:06:58 +05:30
parent fcb6c4c43c
commit fa709a6cde
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/server/appsmith-server/src/main/java/com/appsmith/server/services/AnalyticsService.java
View File

@ -65,13 +65,13 @@ public class AnalyticsService {
return userMono return userMono
.map(user -> { .map(user -> {
// In case the user is anonymous, return as is without raising the event. // In case the user is anonymous, don't raise an event, unless it's a signup event.
if (user.isAnonymous()) { if (user.isAnonymous() && !(object instanceof User && event == AnalyticsEvents.CREATE)) {
return object; return object;
} }
HashMap<String, String> analyticsProperties = new HashMap<>(); HashMap<String, String> analyticsProperties = new HashMap<>();
analyticsProperties.put("id", object.getId()); analyticsProperties.put("id", object instanceof User ? ((User) object).getUsername() : object.getId());
analyticsProperties.put("object", object.toString()); analyticsProperties.put("object", object.toString());
analytics.enqueue( analytics.enqueue(