FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add integration attributes to sdk by v-mamaya · Pull Request #26 · mParticle/mparticle-python-sdk · GitHub

Add integration attributes to sdk - #26

Merged
alexs-mparticle merged 2 commits into
mParticle:masterfrom
v-mamaya:integration-attributes
Jun 20, 2023
Merged

Add integration attributes to sdk#26
alexs-mparticle merged 2 commits into
mParticle:masterfrom
v-mamaya:integration-attributes

Conversation

Copy link
Copy Markdown
Contributor

Summary

Added support for integration attributes in case they are needed to be included using the Python SDK

Testing Plan

  • Was this tested locally?
  • I tested using unit tests, by asigning a dictionary with the proper structure to the batch object and checking the resulting output when it is assigned
  • I've sent some batches via Custom Feed using the Python SDK and verified the proper integration attributes appear on the batch in Livestream

v-mamaya marked this pull request as ready for review June 15, 2023 18:25

Copy link
Copy Markdown
Contributor Author

hey @alexs-mparticle, can you take a look and let me know if we should change something?

alexs-mparticle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Looks good. Only one style issue. I can release this once the change is made.

Comment thread example_usage.py Outdated
Co-authored-by: Alex S <49695018+alexs-mparticle@users.noreply.github.com>
v-mamaya requested a review from alexs-mparticle June 20, 2023 14:23
alexs-mparticle merged commit 6772fc2 into mParticle:master Jun 20, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL