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

Event metric name v2.x #109

Merged
merged 1 commit into from
Jan 12, 2024
Merged

Event metric name v2.x #109

merged 1 commit into from
Jan 12, 2024

Conversation

evan-masseau
Copy link
Contributor

@evan-masseau evan-masseau commented Jan 10, 2024

Description

Finishes the removal of old event names by deleting EventType and renaming some argument labels.
See #107 for first set of changes.

Check List

  • Are you changing anything with the public API? Yes
  • Are your changes backwards compatible with previous SDK Versions? NOPE
  • Have you tested this change on real device? Yes
  • Have you added unit test coverage for your changes? Yes
  • Have you verified that your changes are compatible with all Android versions the SDK currently supports? Yes

Changelog / Code Overview

Remove deprecated EventType

Test Plan

Unit tests and test app

Related Issues/Tickets

CHNL-4084

@evan-masseau evan-masseau changed the base branch from master to event-metric-name-v1.x January 10, 2024 19:54
@evan-masseau evan-masseau changed the base branch from event-metric-name-v1.x to rel/2.0.0 January 11, 2024 20:51
@evan-masseau evan-masseau marked this pull request as ready for review January 11, 2024 20:54
@evan-masseau evan-masseau requested a review from a team as a code owner January 11, 2024 20:54
@evan-masseau evan-masseau requested review from ajaysubra and removed request for a team January 11, 2024 20:54
@evan-masseau evan-masseau added the major-release For PRs that contain breaking changes and belong in a major release label Jan 11, 2024
Include API breaking change to rename argument labels now
@evan-masseau evan-masseau merged commit aa93341 into rel/2.0.0 Jan 12, 2024
2 checks passed
@evan-masseau evan-masseau deleted the event-metric-name-v2.x branch January 12, 2024 16:07
evan-masseau added a commit that referenced this pull request Jan 12, 2024
And rename argument/property labels for event type -> metric

Co-authored-by: Evan Masseau <>
evan-masseau pushed a commit that referenced this pull request Jan 24, 2024
* master:
  Run staleness check during ET work hours
  Release version 2.0.0
  Documentation: top-level redirect for github pages (#119)
  Pre-release version 2.0.0-alpha.3
  Required for android to target jvm 11
  Alpha release 2.0.0-alpha.2
  Target SDK 34 and other dependency upgrades (#115)
  Increment version for pre-release. Leave dokka unchanged, we won't publish docs for prerelease.
  Remove deprecated EventType and update migration guide. (#109)
  Change Event.Value to a double (#110)

# Conflicts:
#	build.gradle
#	docs/index.html
#	versions.gradle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major-release For PRs that contain breaking changes and belong in a major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants