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

feat(teaching): cache course files with readable names #443

Merged
merged 7 commits into from
Feb 26, 2024

Conversation

umbopepato
Copy link
Member

@umbopepato umbopepato commented Feb 12, 2024

Closes #139

  • Save course files with a combination of the original name, the id and the path inside the course folder
  • Switch to DocumentDirectoryPath for local caching in order to make the cache visible to the user on iOS
  • Add a migration to automatically move old cached files to the new structure
  • Add a logic to automatically update cached files when the name/path changes remotely

@umbopepato umbopepato requested a review from a team as a code owner February 12, 2024 19:45
@github-actions github-actions bot added the cc-feat PR includes feature commit label Feb 12, 2024
@github-actions github-actions bot added cc-feat PR includes feature commit and removed cc-feat PR includes feature commit labels Feb 15, 2024
@github-actions github-actions bot added cc-feat PR includes feature commit and removed cc-feat PR includes feature commit labels Feb 19, 2024
@github-actions github-actions bot added cc-feat PR includes feature commit and removed cc-feat PR includes feature commit labels Feb 24, 2024
@github-actions github-actions bot added cc-feat PR includes feature commit and removed cc-feat PR includes feature commit labels Feb 26, 2024
@Bri74 Bri74 merged commit ae89aeb into main Feb 26, 2024
9 checks passed
@Bri74 Bri74 deleted the feature/course-files-readable-names branch February 26, 2024 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cc-feat PR includes feature commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Assign downloaded course files the same name (and path) they have inside the course
3 participants