Skip to content
Snippets Groups Projects
Unverified Commit 1fa2570a authored by Brian Beggs's avatar Brian Beggs Committed by GitHub
Browse files

Merge pull request #19459 from edx/amannan/DE-1171

Add revenue field to completed order event
parents bd76eee0 ac81c5ea
No related branches found
No related tags found
No related merge requests found
...@@ -524,6 +524,9 @@ class Order(models.Model): ...@@ -524,6 +524,9 @@ class Order(models.Model):
segment.track(self.user.id, event_name, { segment.track(self.user.id, event_name, {
'orderId': self.id, 'orderId': self.id,
'total': str(self.total_cost), 'total': str(self.total_cost),
# For Rockerbox integration, we need a field named revenue since they cannot parse a field named total.
# TODO: DE-1188: Remove / move Rockerbox integration code.
'revenue': str(self.total_cost),
'currency': self.currency, 'currency': self.currency,
'products': [item.analytics_data() for item in orderitems] 'products': [item.analytics_data() for item in orderitems]
}) })
......
...@@ -277,6 +277,7 @@ class OrderTest(ModuleStoreTestCase): ...@@ -277,6 +277,7 @@ class OrderTest(ModuleStoreTestCase):
'orderId': 1, 'orderId': 1,
'currency': 'usd', 'currency': 'usd',
'total': '40.00', 'total': '40.00',
'revenue': '40.00', # value for revenue field is same as total.
'products': [ 'products': [
{ {
'sku': u'CertificateItem.honor', 'sku': u'CertificateItem.honor',
...@@ -876,6 +877,7 @@ class CertificateItemTest(ModuleStoreTestCase): ...@@ -876,6 +877,7 @@ class CertificateItemTest(ModuleStoreTestCase):
'orderId': 1, 'orderId': 1,
'currency': 'usd', 'currency': 'usd',
'total': '40.00', 'total': '40.00',
'revenue': '40.00', # value for revenue field is same as total.
'products': [ 'products': [
{ {
'sku': u'CertificateItem.verified', 'sku': u'CertificateItem.verified',
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment