Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: added identify and attributes #163

Merged

Conversation

Shahroz16
Copy link
Collaborator

closes: https://linear.app/customerio/issue/MBL-631/identify-and-attributes

changes:

  • Method signature
  • Ability to add attributes for identified and anonymous profile
  • Tests updates

@Shahroz16 Shahroz16 changed the base branch from main to feature/data-pipelines-support November 7, 2024 08:17
Copy link

github-actions bot commented Nov 7, 2024

Sample app builds 📱

Below you will find the list of the latest versions of the sample apps. It's recommended to always download the latest builds of the sample apps to accurately test the pull request.


  • amiapp_flutter: 163.7.0 (28850041)

@Shahroz16 Shahroz16 requested a review from mrehan27 November 7, 2024 08:17
Copy link
Contributor

@mrehan27 mrehan27 left a comment

Choose a reason for hiding this comment

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

Looks good. Would appreciate if you could revert the formatting updates as mentioned in other comment.

}

CustomerIO.shared.track(name: name, data: attributes)
guard let name = params[Keys.Tracking.eventName] as? String
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it possible to revert these formatting changes for commented code? As it could be a bit challenging when merging as I'm working in same file in other ticket.

@mrehan27
Copy link
Contributor

mrehan27 commented Nov 7, 2024

Also, please make sure the failing tests are fixed before merging.

@Shahroz16 Shahroz16 merged commit 40c14f0 into feature/data-pipelines-support Nov 7, 2024
6 checks passed
@Shahroz16 Shahroz16 deleted the mbl-631-identify-and-attributes branch November 7, 2024 18:14
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