-
Notifications
You must be signed in to change notification settings - Fork 37
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
Reader SDK version bump to 1.7.4 #232
base: master
Are you sure you want to change the base?
Conversation
do you also need to update the
|
Nit: if we're going to bother with the overall-commit template, we should fill it in. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left several comments, only one of them blocking: IIUC what package.json says, then the comment in android/app/build.gradle is wrong (and should probably just go away entirely, but is definitely wrong)
...k-react-native-quickstart/android/app/src/main/java/com/rnreadersdksample/MainApplication.kt
Outdated
Show resolved
Hide resolved
@@ -1,13 +1,17 @@ | |||
## Changelog | |||
|
|||
### v1.4.6 Sep 18, 2024 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the latest entry in the changelog should be:
v1.7.7 Oct 16, 2024
- Upgrade Reader SDK version support to 1.7.4 for iOS
- Support CompileSDK and targetSDK 34 on Android
So replace your changes adding ### v1.4.6 Sep 18, 2024
with that.
### v1.4.5 Apr 25, 2024 | ||
|
||
* Upgrade Reader SDK version support to 1.7.2 for iOS | ||
|
||
### v1.4.4 Aug 16, 2023 | ||
|
||
* Upgrade Reader SDK version support to 1.7.5 for Android | ||
* Support CompileSDK and targetSDK 33 on Android | ||
* Support CompileSDK and targetSDK 34 on Android |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revert this change, see comment above for v1.7.7 Oct 16, 2024 entry.
Summary
Related issues
Fix #
Changelog
Test Plan