Don't need to force everything to string when serializing usage reports

Author: shana

src/GitHub.Exports/Services/MetricsService.cs

@@ -104,29 +104,13 @@ static StringContent SerializeRequest(UsageModel model)
             var serializer = new SimpleJsonSerializer();
             var dictionary = new Dictionary<string, object>
             {
-                {ToJsonPropertyName("Dimensions"), ToStringDictionary(model.Dimensions) },
-                {ToJsonPropertyName("Measures"), ToObjectDictionary(model.Measures) }
+                {ToJsonPropertyName("Dimensions"), ToModelDictionary(model.Dimensions) },
+                {ToJsonPropertyName("Measures"), ToModelDictionary(model.Measures) }
             };
             return new StringContent(serializer.Serialize(dictionary), Encoding.UTF8, "application/json");
         }
 
-        static Dictionary<string, string> ToStringDictionary(object model)
-        {
-            var dict = new Dictionary<string, string>();
-            var type = model.GetType();
-
-            foreach (var prop in type.GetProperties())
-            {
-                if (prop.PropertyType.IsValueType || prop.PropertyType == typeof(string))
-                {
-                    dict.Add(ToJsonPropertyName(prop.Name), prop.GetValue(model).ToString());
-                }
-            }
-
-            return dict;
-        }
-
-        static Dictionary<string, object> ToObjectDictionary(object model)
+        static Dictionary<string, object> ToModelDictionary(object model)
         {
             var dict = new Dictionary<string, object>();
             var type = model.GetType();
@@ -137,11 +121,25 @@ static Dictionary<string, object> ToObjectDictionary(object model)
                 {
                     dict.Add(ToJsonPropertyName(prop.Name), prop.GetValue(model));
                 }
+                else
+                {
+                    var value = prop.GetValue(model);
+
+                    if (value == null)
+                    {
+                        dict.Add(ToJsonPropertyName(prop.Name), value);
+                    }
+                    else
+                    {
+                        dict.Add(ToJsonPropertyName(prop.Name), ToModelDictionary(value));
+                    }
+                }
             }
 
             return dict;
         }
 
+
         /// <summary>
         /// Convert from PascalCase to camelCase.
         /// </summary>