Skip to content

[MBL-19871][All] Fix media attachments not appearing in Inbox#3611

Merged
kristofnemere merged 2 commits intomasterfrom
MBL-19871
Mar 31, 2026
Merged

[MBL-19871][All] Fix media attachments not appearing in Inbox#3611
kristofnemere merged 2 commits intomasterfrom
MBL-19871

Conversation

@kristofnemere
Copy link
Copy Markdown
Contributor

@kristofnemere kristofnemere commented Mar 27, 2026

Test plan:

  1. Open the Canvas app (Student, Teacher, or Parent)
  2. Navigate to Inbox and open a message that contains a media comment attachment (recorded via "Record an Audio or Video Comment")
  3. Verify the media attachment is displayed in the message
  4. Tap the attachment and verify it opens in a video player

refs: MBL-19871
affects: Student, Teacher, Parent
release note: Fixed an issue where media attachments sent via Inbox were not visible or playable on Android devices.

  • Follow-up e2e test ticket created or not needed
  • Tested in dark mode
  • Tested in light mode
  • Test in landscape mode and/or tablet
  • A11y checked
  • Approve from product

kristofnemere and others added 2 commits March 27, 2026 14:00
- Fix MIME type resolution in OpenMediaAsyncTaskLoader: when contentResolver
  returns application/octet-stream (no file extension), fall back to the known
  mimeType from the API/response headers instead of only handling PDFs
- Normalize application/mp4 to video/mp4 so Android video players can open
  media comment attachments served by Kaltura/media download URLs
- Rename OpenMediaComment action to OpenMediaAttachment for consistency

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 27, 2026

🧪 Unit Test Results

✅ 📱 Parent App

  • Tests: 315 total, 0 failed, 0 skipped
  • Duration: 35.530s
  • Success Rate: 100%

✅ 📱 Student App

  • Tests: 1252 total, 0 failed, 0 skipped
  • Duration: 0.000s
  • Success Rate: 100%

✅ 📱 Teacher App

  • Tests: 373 total, 0 failed, 0 skipped
  • Duration: 35.100s
  • Success Rate: 100%

✅ 🌅 Horizon

  • Tests: 790 total, 0 failed, 0 skipped
  • Duration: 44.631s
  • Success Rate: 100%

✅ 📦 Submodules

  • Tests: 3344 total, 0 failed, 0 skipped
  • Duration: 56.718s
  • Success Rate: 100%

📊 Summary

  • Total Tests: 6074
  • Failed: 0
  • Skipped: 0
  • Status: ✅ All tests passed!

Last updated: Fri, 27 Mar 2026 13:17:41 GMT

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 27, 2026

📊 Code Coverage Report

ℹ️ Student

  • PR Coverage: 42.64%
  • Master Coverage: N/A

ℹ️ Teacher

  • PR Coverage: 25.37%
  • Master Coverage: N/A

ℹ️ Pandautils

  • PR Coverage: 23.66%
  • Master Coverage: N/A

@github-actions
Copy link
Copy Markdown
Contributor

Parent Install Page

@github-actions
Copy link
Copy Markdown
Contributor

Student Install Page

@github-actions
Copy link
Copy Markdown
Contributor

Teacher Install Page

Copy link
Copy Markdown
Contributor

@kdeakinstructure kdeakinstructure left a comment

Choose a reason for hiding this comment

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

QA 👍

@kristofnemere kristofnemere merged commit a997e5c into master Mar 31, 2026
69 of 73 checks passed
@kristofnemere kristofnemere deleted the MBL-19871 branch March 31, 2026 10:23
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.

4 participants