Skip to content

Conversation

amirsaam
Copy link
Member

  • New Cacher class has been introduced in Cacher.swift that uses DataCache package
  • Removed ImageCache class and rewrote the logic and reworked all usages of it
  • Exposed getITunesData function so any part of PlayCover can use it as a function as it should not be in a class that is unrelated to its functionality
  • Online Images now are being cached via CachedAsyncImage package that uses URLCache as memory
  • Installed app icons extracted as NSImage and will be updated on each update (fixes old ImageCache bug)

@amirsaam amirsaam mentioned this pull request Dec 24, 2022
@IsaacMarovitz IsaacMarovitz requested a review from Depal1 December 28, 2022 20:35
@amirsaam amirsaam requested review from KhoraLee and removed request for Depal1 December 29, 2022 11:11
Copy link
Contributor

@KhoraLee KhoraLee left a comment

Choose a reason for hiding this comment

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

LGTM

@Depal1 Depal1 merged commit da51290 into PlayCover:develop Dec 29, 2022
@amirsaam amirsaam deleted the New-Cache-System branch December 29, 2022 20:06
IsaacMarovitz pushed a commit to IsaacMarovitz/PlayCover that referenced this pull request Mar 11, 2023
* New Cache System

* Update README.md

* Update Localizable.strings

* Update Cacher.swift

* Update MenuBarView.swift

* load time fix

* code cleanup

* Resolving Reviews

* I pushed Code Signings again :iaminpain:
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