Skip to content

Update findings and recommendations for storage optimization - #12

Open
kameron-ctrl wants to merge 1 commit into
contributor-catalyst:mainfrom
kameron-ctrl:main
Open

Update findings and recommendations for storage optimization#12
kameron-ctrl wants to merge 1 commit into
contributor-catalyst:mainfrom
kameron-ctrl:main

Conversation

@kameron-ctrl

Copy link
Copy Markdown
Contributor

Updated the findings and recommendations section regarding offline reference functionality, file compression, and search feature considerations.

Updated the findings and recommendations section regarding offline reference functionality, file compression, and search feature considerations.
Comment thread p5jswebsite-2026.md
- [Kam will need help with this] - What we recommend next and why
- Good place to mention what would be a reasonable size for the .zip
- Also a good place to share thoughts on including all images, search functionality, language support, etc.
- For a smaller downloadable zip we must discuss what all we want in the zip. The search feature is expendable especiallly since the filter feature already exists. Also the assets is taking up a great

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great work and investigation into the further viability of ffmpeg Kam! Earlier in the document @riahdollxo suggested leaving the search function in because despite it's lack of functionality the size is negligible enough to make removing it more of a hassle than the problems that it may cause. I'll leave it up to you to decide which route is best, perhaps after a brief discussion with @riahdollxo . I just would want to make sure both this section on line 47 and the section on line 38 present a united front on the suggested handling of the search function.

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