Skip to content

Miduo/71 media failing linux#72

Merged
gjwgit merged 16 commits into
devfrom
miduo/71_media_failing_Linux
Apr 24, 2026
Merged

Miduo/71 media failing linux#72
gjwgit merged 16 commits into
devfrom
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

@gjwgit gjwgit self-requested a review April 14, 2026 06:43
Copy link
Copy Markdown
Owner

@gjwgit gjwgit left a comment

Choose a reason for hiding this comment

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

Thanks @Miduo666

Audio and video working on Linux

Except I see this overflow:

Image

Enlarging screen fixes it but might need a Flexible() or something around the widget.

@Miduo666
Copy link
Copy Markdown
Collaborator Author

Thanks @Miduo666

Audio and video working on Linux

Except I see this overflow:

Image Enlarging screen fixes it but might need a Flexible() or something around the widget.

Spot on! It was indeed an overflow issue. I've fixed it.

@gjwgit
Copy link
Copy Markdown
Owner

gjwgit commented Apr 24, 2026

Still an issue but merge this AI update.

image

@gjwgit gjwgit merged commit 9ec89db into dev Apr 24, 2026
27 of 30 checks passed
@gjwgit gjwgit deleted the miduo/71_media_failing_Linux branch April 24, 2026 10:18
@Miduo666
Copy link
Copy Markdown
Collaborator Author

Still an issue but merge this AI update.

image

That’s strange — I tested this on Android earlier and it appeared to work fine. I’ll take another look.

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