Merged
Conversation
* Update top level README with updated 2.0 instructions * Update top level README with updated 2.0 instructions * Update repo URL * Update final test app to use 2.0 registration and log service Update XML files to have xml as absolute first element Update build dependencies to use 2.0 versions of extensions Remove jetifier * Update start test app to use 2.0 registration and log service Update XML files to have xml as absolute first element Update build dependencies to use 2.0 versions of extensions Remove jetifier * Update main readme with new doc links and updated instructions * Add deprecated warning to registerExtension API * Update styling * Replace outdated doc link * Update instruction to be theme agnostic Add alternative gradle sync instructions * Consolidate documentation links in main README Update formatting * Update styling * Update readme and api reference Replace all usages of acronym * Remove link to lifecycle implementation in tutorial * update tutorial app MainApp LOG_SOURCE * Align main README with edgeconsent layout * Update years for copyright and license * Update both getting started guides * Update assurance connection instructions * Update tutorial app log statements Remove api deeplink * Update tutorial app logs * Update api doc url * Update getting started example registration Update doc readme with test app getting started * Fix swift syntax to java
* Edge test app kt * create util lib to share common code across test apps * Format files * Review impl
* WIP kotlin api updates * WIP kotlin api updates * Remove kotlin converted class examples * Group examples by language Add kotlin example for Experience Event * Remove interface required methods * Update gitbook link to main core repo * Align readme table with edgeconsents current wording * Update api docs according to feedback
* Update to latest dependencies * More dependencies cleanup
addb
approved these changes
Jan 31, 2023
Codecov Report
@@ Coverage Diff @@
## staging #40 +/- ##
=============================================
- Coverage 83.28% 79.16% -4.12%
+ Complexity 367 359 -8
=============================================
Files 29 29
Lines 1555 1555
Branches 219 219
=============================================
- Hits 1295 1231 -64
- Misses 161 223 +62
- Partials 99 101 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
|
cacheung
approved these changes
Jan 31, 2023
* Update header spacing * Remove whitespace after headers
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
4 participants
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.
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: