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

feat: Migration to modern restoration API #1187

Merged
merged 87 commits into from
Jul 1, 2024

Conversation

adrien-coye
Copy link
Contributor

@adrien-coye adrien-coye commented May 23, 2024

Abstract

Migrating to UISceneDelegate, as the requirement for this is iOS13.
This will be the occasion to split navigation code away from the AppDelegate.
Also checking old restoration code while migrating it.
State restoration no longer has a stand-alone code path, but instead use routing code to replay navigation commands.
Navigation was reworked to make good use of async await.

Global TODOs:

@adrien-coye adrien-coye force-pushed the releaseAPIV3-restorationScene branch from 50237ab to 05a26c7 Compare May 23, 2024 13:03
Copy link
Contributor Author

@adrien-coye adrien-coye left a comment

Choose a reason for hiding this comment

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

I did a self review

kDrive/AppDelegate.swift Outdated Show resolved Hide resolved
kDrive/AppDelegate.swift Outdated Show resolved Hide resolved
kDrive/AppDelegate.swift Outdated Show resolved Hide resolved
kDrive/AppRouter.swift Outdated Show resolved Hide resolved
kDrive/UI/Controller/LockedAppViewController.swift Outdated Show resolved Hide resolved
kDrive/UI/Controller/Menu/MenuViewController.swift Outdated Show resolved Hide resolved
kDrive/UI/Controller/Menu/MenuViewController.swift Outdated Show resolved Hide resolved
Copy link
Contributor Author

@adrien-coye adrien-coye left a comment

Choose a reason for hiding this comment

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

did another self assesment

kDrive/AppRouter.swift Outdated Show resolved Hide resolved
kDrive/Resources/Info.plist Show resolved Hide resolved
kDrive/SceneDelegate.swift Outdated Show resolved Hide resolved
kDrive/SceneDelegate.swift Outdated Show resolved Hide resolved
kDrive/AppRouter.swift Outdated Show resolved Hide resolved
kDriveCore/Data/Cache/AccountManager.swift Show resolved Hide resolved
kDriveCore/Utils/Sentry/SentryDebug.swift Show resolved Hide resolved
@adrien-coye adrien-coye marked this pull request as ready for review May 24, 2024 10:35
kDrive/AppDelegate.swift Outdated Show resolved Hide resolved
@adrien-coye adrien-coye mentioned this pull request May 28, 2024
5 tasks
@adrien-coye adrien-coye marked this pull request as ready for review July 1, 2024 06:42
Copy link

sonarcloud bot commented Jul 1, 2024

@adrien-coye adrien-coye merged commit ec790cd into releaseAPIV3 Jul 1, 2024
3 checks passed
@adrien-coye adrien-coye deleted the releaseAPIV3-restorationScene branch July 1, 2024 06:45
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.

None yet

2 participants