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

Reader SDK version bump to 1.7.4 #232

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Nilay-squareup
Copy link
Collaborator

Summary

Related issues

Fix #

Changelog

  • message

Test Plan

@plinio-square
Copy link
Collaborator

do you also need to update the package.json file to update the version of the plugin?

@fka3
Copy link
Contributor

fka3 commented Sep 18, 2024

Nit: if we're going to bother with the overall-commit template, we should fill it in.

CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

@fka3 fka3 left a 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)

RNReaderSDK.podspec Outdated Show resolved Hide resolved
package.json Show resolved Hide resolved
### 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
* Upgrade Reader SDK version support to 1.7.7 for Android
Copy link
Member

Choose a reason for hiding this comment

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

I don't think you should be updating the previous changelog. All of the changes should be for the newest entry.

@@ -1,13 +1,17 @@
## Changelog

### v1.4.6 Sep 18, 2024
Copy link
Member

Choose a reason for hiding this comment

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

This should be v1.7.7 right? That's what this is updating it to?

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.

4 participants