Skip to content
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

Fix media in offline rich content #2977

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

vargaat
Copy link
Contributor

@vargaat vargaat commented Nov 20, 2024

What's new?

  • URLs in rich content pointing to local offline content were modified to be referenced via relative urls instead of absolute ones.
  • I simplified the HTMLDownloadInteractor entity to be easier to understand what it does.

refs: MBL-18070
affects: Student
release note: Fixed images and Studio videos not being available for offline mode after an app update.

test plan:

  • Check the ticket for a detailed description on the issue.
  • Sync rich content with images and studio videos.
  • Go offline and go to the rich content.
  • They should appear just as expected.
  • Check the offline html content with safari or with a breakpoint in CoreWebView.loadContent (look for the rawHtmlValue variable).
  • Urls should be all relative without the application's UUID container ID in it.
  • Check out the branch and install the app into the same phone to simulate an app update.
  • Start the app in offline mode, rich content should appear just as before.
  • Note: we can't test this with other builds because the base url for the offline webview have changed and other builds point to a different one.

Checklist

  • Follow-up e2e test ticket created
  • A11y checked
  • Tested on phone
  • Tested on tablet
  • Tested in dark mode
  • Tested in light mode
  • Approve from product

refs: MBL-18070
affects: Student
release note: Fixed images and Studio videos not being available for offline mode after an app update.

test plan:
- Sync rich content with images and studio videos.
- Go offline and go to the rich content.
- They should appear just as expected.
- Check the offline html content with safari or with a breakpoint in CoreWebView.loadContent (look for the rawHtmlValue variable).
- Urls should be all relative without the application's UUID container ID in it.
- Check out the branch and install the app into the same phone to simulate an app update.
- Start the app in offline mode, rich content should appear just as before.
- Note: we can't test this with other builds because the base url for the offline webview have changed and other builds point to a different one.
@inst-danger
Copy link
Contributor

inst-danger commented Nov 20, 2024

Warnings
⚠️ One or more files are below the minimum test coverage 50%

Release Note:

Fixed images and Studio videos not being available for offline mode after an app update.

Affected Apps: Student

MBL-18070

Coverage New % Master % Delta
Canvas iOS 90.19% 90.17% 0.03%
Core/Core/Planner/CalendarMain/ViewModel/PlannerViewModel.swift 0% 0% 0%
Core/Core/Planner/CalendarMain/Model/CalendarWeek.swift 0% 0% 0%
Core/Core/Planner/CalendarMain/View/Calendar/CalendarCardInteractionState.swift 0% 0% 0%
Core/Core/Planner/CalendarMain/Model/PlannablesInteractor.swift 0% 0% 0%
Core/Core/Planner/CalendarMain/View/Calendar/ViewPreferences.swift 0% 0% 0%
Core/Core/Planner/CalendarMain/Model/CalendarDay.swift 0% 0% 0%
Core/Core/Planner/CalendarMain/Model/CalendarMonth.swift 0% 0% 0%
Core/Core/Planner/CalendarMain/View/Calendar/CalendarUtils.swift 0% 0% 0%
Core/Core/Assignments/AssignmentList/ViewModel/AssignmentListViewModel.swift 47.8% 47.8% 0%

Generated by 🚫 dangerJS against c8a12c6

@vargaat vargaat self-assigned this Nov 20, 2024
@inst-danger
Copy link
Contributor

Student Build QR Code:

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