fix: change the types of event revenue and value fields

optimizely / swift-sdk

Swift SDK for Optimizely Feature Experimentation and Optimizely Full Stack (legacy)

https://www.optimizely.com/products/full-stack/

Apache License 2.0

21 stars 30 forks source link

fix: change the types of event revenue and value fields #467

Closed jaeopt closed 2 years ago

jaeopt commented 2 years ago

Summary

The revenue and value fields of events are changed to integer and float types.

These changes are for internal only. There will be no change visible to users.
They were implemented as AttributeValue type (the core types were validated when extracted).

Test plan

Pass existing unit and FSC tests

Issues

FSSDK-8562
https://github.com/optimizely/swift-sdk/issues/244

coveralls commented 2 years ago

Coverage remained the same at 95.735% when pulling 51060c835bea348f5827dc31ad1dc7ed2475f3c8 on jae/FSSDK-8562 into 726ee35d2ded53315532e784e14fbe37230a02cb on master.