Skip to content

Add x-native-sdk header in CDSHandler - #22

Merged
Petrakeas merged 1 commit into
develfrom
petrakeas/x-native-sdk_header
Jun 15, 2021
Merged

Petrakeas merged 1 commit into
develfrom
petrakeas/x-native-sdk_header

Conversation

@Petrakeas

Copy link
Copy Markdown
Contributor

The build.gradle of "common" module can now generate a
BuildProperites.java file that exposes some gradle properties
to Java.

BuildProperties#getSDKVersion() is used in CDSHandler#addHeaders() so that
"x-native-sdk" -> "mobile/android/{TXNative.version}" header is added when pushing
or pulling strings.

CLI used to get the CLI version from the "Implementation-Version" property of the jar
file. We now use BuildProperties#getCLIVersion() to get the version.

The build.gradle of "common" module can now generate  a
BuildProperites.java file that exposes some gradle properties
to Java.

BuildProperties#getSDKVersion() is used in CDSHandler#addHeaders() so that
"x-native-sdk" -> "mobile/android/{TXNative.version}" header is added when pushing
or pulling strings.

CLI used to get the CLI version from the "Implementation-Version" property of the jar
file. We now use BuildProperties#getCLIVersion() to get the version.

@dbendilas dbendilas left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks great! Feel free to merge

@Petrakeas
Petrakeas merged commit 350dc76 into devel Jun 15, 2021
@Petrakeas
Petrakeas deleted the petrakeas/x-native-sdk_header branch June 15, 2021 09:04
@wyngarde wyngarde mentioned this pull request Jun 22, 2021
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