Skip to content

Miduo/71 media failing linux#72

Open
Miduo666 wants to merge 16 commits intodevfrom
miduo/71_media_failing_Linux
Open

Miduo/71 media failing linux#72
Miduo666 wants to merge 16 commits intodevfrom
miduo/71_media_failing_Linux

Conversation

@Miduo666
Copy link
Copy Markdown
Collaborator

@Miduo666 Miduo666 commented Apr 7, 2026

Pull Request Details

Description

Resolved Linux audio/video playback issues by integrating video_player_media_kit.

Debug Mode: Added a new DEBUG mode for easier development and troubleshooting.

Cache Logic Fix: Resolved a bug where stale data was being read from the cache even after updates.

Media Handling: Improved logic to check for encrypted media before playback to prevent crashes/errors.

These changes are necessary to provide cross-platform support (Linux), ensure data consistency between the local cache and the server, and enhance the overall robustness of media playback.

Related Issues

#71
#66

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How To Test?

Media Playback: Play an encrypted media file. This simultaneously verifies:

The fix for Linux audio/video playback (Issue #71).

The new pre-playback logic that checks for encryption/file compatibility.

Debug Mode: Navigate to Settings and toggle the Debug mode to verify the UI and backend flag implementation.

Cache Refresh: Update a data entry and verify that the app reads the new data rather than stale cache, ensuring the updated cache logic works as expected.

Checklist

  • Screenshots included in linked issue #
  • Changes adhere to the style and coding guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules
  • The update contains no confidential information
  • The update has no duplicated content
  • No lint check errors are related to these changes (make prep or flutter analyze lib)
  • Integration test dart test output or screenshot included in issue #
  • I tested the PR on these devices:
    • Android
    • iOS
    • Linux
    • MacOS
    • Windows
    • Web
  • I have identified reviewers
  • The PR has been approved by reviewers

Finalising

  • Merge dev into the this branch
  • Resolve any conflicts
  • Add a one line summary into the CHANGELOG.md
  • Push to the git repository and review
  • Merge the PR into dev

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