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

Remove the trial license key requirement #403

Merged
merged 5 commits into from
May 26, 2021

Conversation

radazzouz
Copy link
Contributor

@radazzouz radazzouz commented May 25, 2021

Details

⚠️ Before reviewing this PR, please take a look at https://github.com/PSPDFKit/PSPDFKit-Website/pull/5233 first. ⚠️

The new getting started instructions from the README are very similar to the ones from https://pspdfkit.com/getting-started/mobile/?frontend=react-native&react-native-platform=android&project=new-project

Release Notes

  • Remove the trial license key requirement.
  • Update to PSPDFKit for Android version 6.6.2.
  • Update the README to clarify the Requirements section.
  • Update the README to simplify the Getting Started section.

Acceptance Criteria

  • Test the Catalog and Native Catalog example projects on Android and iOS.
  • Test a newly created project on Android and iOS using the new Getting Started Instructions in the README. When testing, please use yarn add github:PSPDFKit/react-native#rad/remove-trial-license-key-requirement for the fourth step.
  • When approved, right before merging, rebase with master and increment the package version in package.json, package-lock.json, samples/Catalog/package.json, and samples/NativeCatalog/package.json (see example commit: ae02732).
  • Create a new release (and tag) with the new package version (see https://github.com/PSPDFKit/react-native/releases).

@radazzouz radazzouz self-assigned this May 25, 2021
@radazzouz radazzouz marked this pull request as draft May 25, 2021 15:55
@radazzouz radazzouz marked this pull request as ready for review May 25, 2021 16:40
Copy link
Contributor

@steviki steviki left a comment

Choose a reason for hiding this comment

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

iOS changes look good to me!

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
radazzouz and others added 4 commits May 26, 2021 07:46
Co-authored-by: Stefan Kieleithner <stefan.kieleithner@gmail.com>
Co-authored-by: Stefan Kieleithner <stefan.kieleithner@gmail.com>
@radazzouz radazzouz changed the title Remove trial license key requirement Remove the trial license key requirement May 26, 2021
Copy link
Contributor

@amit-nayar amit-nayar left a comment

Choose a reason for hiding this comment

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

Android guide LGTM

ext {
buildToolsVersion = "28.0.3"
- minSdkVersion = 16
+ minSdkVersion = 19
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we not need to set this anymore?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, the minSdkVersion for a newly created project is now 21.

@radazzouz radazzouz merged commit ae764b3 into master May 26, 2021
@radazzouz radazzouz deleted the rad/remove-trial-license-key-requirement branch May 26, 2021 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants