refactor(ui): remove unused image cache system and fix TypeScript compilation #164

Open
anomalist wants to merge 0 commits from remove-image-cache into master
anomalist commented 5 days ago
Owner
  • Remove imageCache Map that only stored null values
  • Remove selectedImageData Blob property (never used)
  • Remove cacheImageData method and related function props
  • Remove handleImageLoad method from ActivityListItem
  • Clean up ImageViewer component props
  • Fix TypeScript compilation by replacing legacy logConsoleAndDb calls
  • Replace logConsoleAndDb with logger.error in deepLinks service
  • Images continue to work via direct URL references

The image cache system was non-functional and only stored null values.
ImageViewer component ignored blob data and only used URLs.
Fixed production build failure caused by undefined logConsoleAndDb function.
Removing dead code improves maintainability without affecting functionality.

- Remove imageCache Map that only stored null values - Remove selectedImageData Blob property (never used) - Remove cacheImageData method and related function props - Remove handleImageLoad method from ActivityListItem - Clean up ImageViewer component props - Fix TypeScript compilation by replacing legacy logConsoleAndDb calls - Replace logConsoleAndDb with logger.error in deepLinks service - Images continue to work via direct URL references The image cache system was non-functional and only stored null values. ImageViewer component ignored blob data and only used URLs. Fixed production build failure caused by undefined logConsoleAndDb function. Removing dead code improves maintainability without affecting functionality.
anomalist added 1 commit 5 days ago
bb357f294a refactor(ui): remove unused image cache system and fix TypeScript compilation
Owner

I approve this!

I approve this!
anomalist added 1 commit 3 days ago
This branch is equal with the target branch.
This branch is out-of-date with the base branch
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.