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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[馃泚] Adding new project properties #705

Merged
merged 3 commits into from Jan 8, 2020
Merged

[馃泚] Adding new project properties #705

merged 3 commits into from Jan 8, 2020

Conversation

eoji
Copy link
Contributor

@eoji eoji commented Jan 8, 2020

馃摬 What

Updating "Project Properties" tracking group.

馃 Why

Another PR to get new events together.

馃洜 How

  • Made project.staticUsdRate not optional.
  • Added new project properties to TrackingClientType.
  • Implemented new project property values in TrackingClient.
  • Updated Lake and Koala tests.

馃憖 See

N/A

馃搵 QA

All project events sent to the Lake 馃挧 should have these properties.
This is a WIP so I'm only hitting staging.

Story 馃摉

Part of NT-654

**Note: ** Tags are not returned in v1 so the value for project_tags will always be null.

Implemented new project property values in TrackingClient.
Updated Lake and Koala tests.
# Conflicts:
#	app/src/test/java/com/kickstarter/libs/LakeTest.kt
@eoji eoji requested a review from ifbarrera January 8, 2020 21:00
@eoji eoji changed the title [馃泚] Adding new project session properties [馃泚] Adding new project properties Jan 8, 2020
@eoji eoji merged commit 4df31eb into master Jan 8, 2020
@eoji eoji deleted the project-properties branch January 8, 2020 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants