Skip to content

UI scene lifecycle migration - #53

Merged
Vabiel merged 6 commits into
masterfrom
UIScene-Lifecycle-migration
Jul 23, 2026
Merged

UI scene lifecycle migration#53
Vabiel merged 6 commits into
masterfrom
UIScene-Lifecycle-migration

Conversation

@Vabiel

@Vabiel Vabiel commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@greymag
greymag requested a review from Copilot July 23, 2026 12:45
@greymag greymag linked an issue Jul 23, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Migrates the iOS side of the plugin to properly support UIKit’s scene-based lifecycle (UISceneDelegate), while also tightening the project’s minimum platform/tooling requirements and fixing a couple of login/init edge cases.

Changes:

  • Raise minimum requirements (Flutter >= 3.38, iOS >= 13) and bump package version to 3.1.0.
  • iOS: add scene URL handling + improve view controller lookup for presenting VK SDK UI; fix init/login completion on authorization failure.
  • Android: treat stored-token permission check failures as a signal to drop invalid tokens (while avoiding reset on connection errors).

Reviewed changes

Copilot reviewed 9 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
README.md Updates documented minimum requirements (Flutter/iOS).
pubspec.yaml Bumps package version and raises Flutter minimum SDK constraint.
ios/flutter_login_vk.podspec Raises iOS deployment target to 13.0.
ios/Classes/SwiftFlutterLoginVkPlugin.swift Adds UIScene URL handling; improves UI presentation controller lookup; fixes init/login completion behavior.
example/pubspec.lock Updates example lockfile to match new package version and SDK constraints.
CHANGELOG.md Adds 3.1.0 release notes describing breaking requirements and fixes.
android/src/main/kotlin/ru/innim/flutter_login_vk/MethodCallHandler.kt Drops stored token when permission check fails (except for IO/connection errors).
.idea/libraries/Dart_SDK.xml Updates local IDE Dart SDK library paths.
.github/workflows/publish.yml Updates publish workflow Flutter version reference.
.github/workflows/analyze.yml Updates CI analyze workflow Flutter version.
.fvmrc Updates pinned Flutter version for FVM.
Files not reviewed (1)
  • .idea/libraries/Dart_SDK.xml: Generated file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pubspec.yaml
Comment thread pubspec.yaml Outdated
@Vabiel
Vabiel merged commit fd54a37 into master Jul 23, 2026
1 check passed
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.

Migrate to UIKit's scene-based lifecycle

3 participants