Skip to content

fix(immersion-kit): Migrate to ImmersionKit API v2#482

Merged
arianneorpilla merged 1 commit intoarianneorpilla:mainfrom
TheAtlasRises:main
Oct 20, 2025
Merged

fix(immersion-kit): Migrate to ImmersionKit API v2#482
arianneorpilla merged 1 commit intoarianneorpilla:mainfrom
TheAtlasRises:main

Conversation

@TheAtlasRises
Copy link
Copy Markdown
Contributor

Migrates from the deprecated ImmersionKit v1 API to the new v2 API.

  • Updates the API endpoint and request/response handling.
  • Correctly constructs image and audio URLs from the new API response format.
  • Adds a local mapping to restore correct casing for media titles.
  • Improves media handling to support multiple images and to correctly set/append images and audio.
  • Replaces the deprecated "ffmpeg_kit_flutter" with "ffmpeg_kit_https_flutter" temporarily. Own fork of "ffmpeg_kit_flutter" might prove necessary.
  • Fix a bug for a dropdown component to handle non-unique or invalid values.

Addresses issues #475 and #449

Important Note:

I had never done any Flutter or Dart before this and was trying to learn an undocumented API on the fly as well, so the quality of this PR code wise is definitely really bad, but as far as my testing goes it fixes the issue for now.

Migrates from the deprecated ImmersionKit v1 API to the new v2 API.
- Updates the API endpoint and request/response handling.
- Correctly constructs image and audio URLs from the new API response format.
- Adds a local mapping to restore correct casing for media titles.
- Improves media handling to support multiple images and to correctly set/append images and audio.
- Replaces the deprecated "ffmpeg_kit_flutter" with "ffmpeg_kit_https_flutter" temporarily, own fork of "ffmpeg_kit_flutter" might prove necessary.
- Fix a bug for a dropdown component to handle non-unique or invalid values.
Copy link
Copy Markdown
Owner

@arianneorpilla arianneorpilla left a comment

Choose a reason for hiding this comment

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

Approved. Thank you @TheAtlasRises!

@arianneorpilla arianneorpilla merged commit 3364999 into arianneorpilla:main Oct 20, 2025
2 checks 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.

2 participants