[Java] Implement the new module for view tracking #182
Conversation
This file contains 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
ArtursKadikis
requested changes
Dec 7, 2023
feat: move convenient functions to utils
arifBurakDemiray
force-pushed
the
new_view_module
branch
from
2701290 to
9120c8d
Compare
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleViews.java
Outdated
Show resolved
Hide resolved
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleViews.java
Outdated
Show resolved
Hide resolved
ArtursKadikis
requested changes
Jan 9, 2024
turtledreams
requested changes
Jan 22, 2024
ArtursKadikis
requested changes
Jan 22, 2024
ijunaid
suggested changes
Jan 30, 2024
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleEvents.java
Outdated
Show resolved
Hide resolved
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleEvents.java
Outdated
Show resolved
Hide resolved
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleViews.java
Outdated
Show resolved
Hide resolved
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleViews.java
Outdated
Show resolved
Hide resolved
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleViews.java
Outdated
Show resolved
Hide resolved
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleViews.java
Outdated
Show resolved
Hide resolved
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleViews.java
Outdated
Show resolved
Hide resolved
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleViews.java
Outdated
Show resolved
Hide resolved
sdk-java/src/main/java/ly/count/sdk/java/internal/ModuleViews.java
Outdated
Show resolved
Hide resolved
arifBurakDemiray
force-pushed
the
new_view_module
branch
from
18035e9 to
171052f
Compare
ijunaid
approved these changes
Feb 1, 2024
* feat: view impl tests * feat: on session began callback * feat: add things to the stop * feat: all bad values and non existing ones * feat: scenario simple flow * fix: simple flow * fix: map usage * fix: rename * feat: segm testing1 * feat: mixed test flow 1 * feat: auto stopped ones * feat: validate segmentation 1 * feat: validate segmentation 2 * feat: validate internal keys * fix: int val check * fix: remove unused var * fix: delete unncessary things * chore: remove whitespcae * chore: whitespace * feat: module views test * feat: integrate views to flow * fix: undo * fix: add missing log * fix: regex pattern * fix: add changelog and deprecate * fix: remove unused import * fix: test thing * fix: device id tests * feat: test id generator * fix: view impl tests * fix: all view ids * feat: additional tests * fix: idx * feat: missing test logic * feat: new way of testing scneario * doc: update comment of scenario id * Update ScenarioManuelViewTests.java * feat: test prefixes * refactor: checkings * Update InternalConfig.java * Update InternalConfig.java * chore: naming sc * refactor: fix PR changes * feat: basic test about global segm * refactor: order * feat: view case proposal * feat: add missing test cases * feat: views module changes * fix: undo * fix: conflict * fix: conflict
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.