Skip to content

fix: data fetching logic and update deprecated packages #138

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

justatipfromthedeadsequoiatree
Copy link

@justatipfromthedeadsequoiatree justatipfromthedeadsequoiatree commented Jun 19, 2025

Summary
This PR addresses issues related to API data fetching and updates outdated or deprecated package usage to ensure compatibility with the latest Flutter and plugin versions.

Changes Made

  1. Fixed invalid API endpoint and removed unused/deprecated APIs.
  2. Updated packages in pubspec.yaml to the latest compatible versions.
  3. Refactored code using deprecated methods to match updated APIs.
  4. Applied minor fixes due to package updates under packages/iridium.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the API data fetching logic and updates deprecated packages to maintain compatibility with the latest Flutter and plugin versions.

  • Fixed API endpoint issues and removed deprecated APIs.
  • Updated package versions and refactored code to align with new API requirements.
  • Adjusted build and configuration files for macOS, iOS, and Windows platforms.

Reviewed Changes

Copilot reviewed 11 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
windows/flutter/generated_plugins.cmake Added the flutter_inappwebview_windows plugin.
windows/flutter/generated_plugin_registrant.cc Registered the new flutter_inappwebview_windows plugin.
pubspec.yaml Updated flutter_inappwebview dependency version from ^5.8.0 to ^6.0.0
packages/iridium/components/shared/lib/src/zip/lazy_zip_decoder.dart Updated constant for zip compression method.
packages/iridium/components/navigator/lib/src/epub/widget/webview_screen_state.dart Changed URL constructor from Uri.parse to WebUri.
macos/Flutter/GeneratedPluginRegistrant.swift Updated plugin imports to reflect new package names.
ios/Runner/AppDelegate.swift Replaced deprecated @UIApplicationMain with @main.
ios/Runner.xcodeproj/xcshareddata/xcschemes/Runner.xcscheme Updated LastUpgradeVersion in scheme file.
ios/Runner.xcodeproj/project.pbxproj Updated build settings to reflect new framework paths and versions.
ios/Flutter/AppFrameworkInfo.plist Raised MinimumOSVersion from 11.0 to 12.0.
Comments suppressed due to low confidence (3)

ios/Runner.xcodeproj/project.pbxproj:267

  • The removal of FMDB and Toast frameworks from the inputPaths may break dependencies if they are still required. Verify that these removals are intentional and that no dependent code references them.
				"${PODS_ROOT}/Target Support Files/Pods-Runner/Pods-Runner-frameworks.sh",

packages/iridium/components/shared/lib/src/zip/lazy_zip_decoder.dart:26

  • Ensure that replacing a.ZipFile.STORE with a.ZipFile.zipCompressionStore maintains the intended compression logic and backward compatibility with existing zip archives.
        final compress = zf.compressionMethod != a.ZipFile.zipCompressionStore;

ios/Flutter/AppFrameworkInfo.plist:24

  • Raising the MinimumOSVersion from 11.0 to 12.0 may affect support for older iOS devices. Confirm that this change aligns with the app's target device support.
  <string>12.0</string>

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.

1 participant