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

Merge SimplyE 3.9.5 release #1630

Closed
wants to merge 20 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
9b8fa8c
Bump version number after release
Feb 15, 2023
002ca14
IOS-567 Integrate R2 via SPM instead of Carthage
ettore Sep 28, 2023
e24e039
IOS-567 Integrate Crashlytics via SPM instead of Carthage
ettore Sep 28, 2023
be0624f
Update README for Xcode 14.3.1 tooling
ettore Oct 11, 2023
8863f7d
Update CI to use macOS-13 and Xcode 14.3
ettore Oct 11, 2023
ae4eaf7
IOS-567 Disable Open eBooks CI checks
ettore Oct 11, 2023
909dc1c
IOS-583 Fix no-drm target to use SPM packages
ettore Oct 11, 2023
07814fd
IOS-583 Add SQLite SPM package to fix no-drm build
ettore Oct 11, 2023
a862333
IOS-583 Update submodule commit refs + schemes to Xcode 14.3
ettore Oct 11, 2023
6596d36
Merge pull request #1624 from NYPL-Simplified/IOS-567/upgrade-tooling
ettore Oct 12, 2023
d4ead68
IOS-578 Suppress Readium 2.5 PublicationServer deprecation warnings
ettore Oct 19, 2023
22f7d20
Bump build number
ExplorerNautilus Oct 28, 2023
8d3abf8
Disable ipa upload to firebase
ExplorerNautilus Nov 8, 2023
8183fb6
Merge pull request #1626 from NYPL-Simplified/iOS-584
ExplorerNautilus Nov 8, 2023
7078bc8
Implement logic to add expiration date to banned books and remove exp…
ExplorerNautilus Nov 17, 2023
7c633ad
Bump build number
ExplorerNautilus Nov 17, 2023
346eab0
Minor refactor on banned books removal
ExplorerNautilus Nov 19, 2023
0a65d96
Remove custom expiration date for banned books
ExplorerNautilus Dec 7, 2023
8ef1a3f
Merge pull request #1627 from NYPL-Simplified/iOS-586
ExplorerNautilus Dec 7, 2023
5402a9d
Bump build number
ExplorerNautilus Dec 8, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions .github/workflows/archive-and-upload.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
- feature/**
jobs:
version-check:
runs-on: macOS-12
runs-on: macOS-13
env:
GITHUB_TOKEN: ${{ secrets.IOS_DEV_CI_PAT }}
steps:
Expand All @@ -20,7 +20,7 @@ jobs:
- name: Check Carthage version
run: command -v carthage && carthage version
- name: Checkout main repo
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
token: ${{ secrets.IOS_DEV_CI_PAT }}
- name: Add Private Repo Auth
Expand All @@ -40,7 +40,7 @@ jobs:
simplye_changed: ${{ steps.detectapp.outputs.simplye_changed }}
openebooks_changed: ${{ steps.detectapp.outputs.openebooks_changed }}
upload-arc:
runs-on: macOS-12
runs-on: macOS-13
needs: version-check
if: needs.version-check.outputs.simplye_changed == '1' || needs.version-check.outputs.openebooks_changed == '1'
env:
Expand All @@ -49,13 +49,13 @@ jobs:
- name: Check available Xcodes
run: ls -la /Applications | grep Xcode
- name: Force Xcode we use for development
run: sudo xcode-select -switch /Applications/Xcode_13.4.1.app
run: sudo xcode-select -switch /Applications/Xcode_14.3.app
- name: Install Firebase Tools
run: npm install -g firebase-tools
- name: Check Firebase Tools version
run: command -v firebase && firebase --version
- name: Checkout main repo and submodules
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
submodules: true
token: ${{ secrets.IOS_DEV_CI_PAT }}
Expand All @@ -72,7 +72,7 @@ jobs:
GITHUB_REPO_OWNER: ${{ github.repository_owner }}
- name: Checkout Adobe RMSDK
if: needs.version-check.outputs.simplye_changed == '1'
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
repository: NYPL-Simplified/DRM-iOS-AdeptConnector
token: ${{ secrets.IOS_DEV_CI_PAT }}
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/non-drm-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,16 @@ on:
pull_request:
jobs:
build:
runs-on: macOS-12
runs-on: macOS-13
steps:
- name: Print System Info
run: uname -a
- name: Check available Xcodes
run: ls -la /Applications | grep Xcode
- name: Force Xcode we use for development
run: sudo xcode-select -switch /Applications/Xcode_13.4.1.app
run: sudo xcode-select -switch /Applications/Xcode_14.3.app
- name: Checkout main repo
uses: actions/checkout@v2
uses: actions/checkout@v3
- name: Set up repo for nonDRM build
run: exec ./scripts/setup-repo-nodrm.sh
env:
Expand Down
16 changes: 8 additions & 8 deletions .github/workflows/unit-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ on:
pull_request:
jobs:
build-and-test:
runs-on: macOS-12
runs-on: macOS-13
if: github.event.pull_request.head.repo.full_name == github.repository
env:
GITHUB_TOKEN: ${{ secrets.IOS_DEV_CI_PAT }}
Expand All @@ -13,14 +13,14 @@ jobs:
- name: Check available Xcodes
run: ls -la /Applications | grep Xcode
- name: Force Xcode we use for development
run: sudo xcode-select -switch /Applications/Xcode_13.4.1.app
run: sudo xcode-select -switch /Applications/Xcode_14.3.app
- name: Checkout main repo and submodules
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
submodules: true
token: ${{ secrets.IOS_DEV_CI_PAT }}
- name: Checkout Adobe RMSDK
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
repository: NYPL-Simplified/DRM-iOS-AdeptConnector
token: ${{ secrets.IOS_DEV_CI_PAT }}
Expand Down Expand Up @@ -52,7 +52,7 @@ jobs:
run: ./scripts/xcode-test.sh simplye
env:
BUILD_CONTEXT: ci
- name: Run Open eBooks tests
run: ./scripts/xcode-test.sh openebooks
env:
BUILD_CONTEXT: ci
# - name: Run Open eBooks tests
# run: ./scripts/xcode-test.sh openebooks
# env:
# BUILD_CONTEXT: ci
2 changes: 1 addition & 1 deletion Axis-iOS
6 changes: 1 addition & 5 deletions Cartfile
Original file line number Diff line number Diff line change
@@ -1,6 +1,2 @@
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseCrashlyticsBinary.json"
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json"
github "NYPL-Simplified/PDFRendererProvider-iOS" "master"
github "stephencelis/SQLite.swift" ~> 0.12.2
github "zxingify/zxingify-objc" ~> 3.6
github "readium/swift-toolkit" "develop"
github "ettore/zxingify-objc" "master"
13 changes: 1 addition & 12 deletions Cartfile.resolved
Original file line number Diff line number Diff line change
@@ -1,13 +1,2 @@
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json" "9.4.0"
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseCrashlyticsBinary.json" "9.4.0"
github "NYPL-Simplified/PDFRendererProvider-iOS" "d63e6ba168cb91617a4aca5e306a92af1dd4497c"
github "cezheng/Fuzi" "3.1.3"
github "dexman/Minizip" "1.4.0"
github "krzyzanowskim/CryptoSwift" "1.5.1"
github "ra1028/DifferenceKit" "1.3.0"
github "readium/GCDWebServer" "3.7.1"
github "readium/swift-toolkit" "5278ddc4c12c69aa6c5946ed4a4e09c0ebf642e3"
github "scinfu/SwiftSoup" "2.4.3"
github "stephencelis/SQLite.swift" "0.12.2"
github "weichsel/ZIPFoundation" "0.9.11"
github "zxingify/zxingify-objc" "3.6.7"
github "ettore/zxingify-objc" "a06018fc4f2120330932548fa8054e2c332dfcac"
2 changes: 1 addition & 1 deletion NYPLAEToolkit
2 changes: 1 addition & 1 deletion RDServices.xcodeproj/project.pbxproj
Original file line number Diff line number Diff line change
Expand Up @@ -389,7 +389,7 @@
118A0E201992B3FD00792DDE /* Project object */ = {
isa = PBXProject;
attributes = {
LastUpgradeCheck = 1340;
LastUpgradeCheck = 1430;
ORGANIZATIONNAME = Readium;
};
buildConfigurationList = 118A0E231992B3FD00792DDE /* Build configuration list for PBXProject "RDServices" */;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?xml version="1.0" encoding="UTF-8"?>
<Scheme
LastUpgradeVersion = "1340"
LastUpgradeVersion = "1430"
version = "1.3">
<BuildAction
parallelizeBuildables = "YES"
Expand Down
22 changes: 4 additions & 18 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ Consequently, [releases](https://github.com/NYPL-Simplified/Simplified-iOS/relea

# System Requirements

- Install Xcode 13.4.1 in `/Applications`, open it and make sure to install additional components if it asks you.
- Install Xcode 14.3.1 in `/Applications`, open it and make sure to install additional components if it asks you.
- Install [Carthage](https://github.com/Carthage/Carthage) 0.38 or newer if you haven't already. Using `brew` is recommended.

# Building without DRM support
Expand Down Expand Up @@ -54,26 +54,12 @@ cd Simplified-iOS #repo root
```
The `scripts` directory contains a number of other scripts to build dependencies more granularly and also to build/test/archive the app from the command line. These scripts are the same used by our CI system. All these scripts must be run from the root of the Simplified-iOS repo, not from the `scripts` directory.

# Building for Readium 2 Integration

Before working on R2 integration, make sure you can build the app by following the steps listed above.

For working on integrating R2 into SimplyE, first clone the following repo as a sibling of `Simplified-iOS` on the file system:
```bash
cd Simplified-iOS/..
git clone https://github.com/readium/swift-toolkit
```
Then rebuild the dependencies:
```bash
cd Simplified-iOS
./scripts/build-carthage-R2-integration.sh
```
Finally, open `SimplifiedR2.workspace` and use the `SimplyE-R2dev` target to build the app.

# Building Open eBooks

Open eBooks is an app primarily targeted toward the education space. It requires DRM. Follow the same steps as indicated above and use the "Open eBooks" Xcode target.

NOTE: the Open eBooks target needs to be upgraded to match SimplyE's.

# Contributing

This codebase follows Google's [Swift](https://google.github.io/swift/) and [Objective-C](https://google.github.io/styleguide/objcguide.xml) style guides, including the use of two-space indentation. More details are available in [our wiki](https://github.com/NYPL-Simplified/Simplified/wiki/Mobile-client-applications#code-style-1).
Expand All @@ -96,7 +82,7 @@ Feature branch names (for features whose development is a month or more): `featu

# License

Copyright © 2015-2022 The New York Public Library, Astor, Lenox, and Tilden Foundations
Copyright © 2015-2023 The New York Public Library, Astor, Lenox, and Tilden Foundations

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
Expand Down
Loading
Loading