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 and Feature dump #324

Merged
merged 22 commits into from
Mar 19, 2023
Merged

Fix and Feature dump #324

merged 22 commits into from
Mar 19, 2023

Conversation

TacticalTechJay
Copy link
Collaborator

This PR should fix #318 and fix #317.

Along with that, it adds the following:

  • approriate url for both webhooks where applicable
  • meta tags and audio preview for audio files. They can't be embedded in discord.
  • compression (content-encoding header) for the raw route, can be applied to the dashboard where files could be previewed.
  • CORE_COMPRESSION_ENABLED, CORE_COMPRESSION_THRESHOLD, and CORE_COMPRESSION_ON_DASHBOARD variables. this will be disabled by default, Their types are boolean, human-to-byte, and boolean. on_dashboard will have compressed file previews being served on the dashboard
  • some npm errors with npm i
  • images resized and fitted to the preview card that's visible both on /dashboard and /files
  • non-previewables having a centered height on said preview card (or what is close to center)

@TacticalTechJay TacticalTechJay requested a review from diced March 13, 2023 04:55
src/server/decorators/rawFile.ts Outdated Show resolved Hide resolved
@diced diced merged commit 5b9b454 into diced:trunk Mar 19, 2023
@diced diced added this to the 3.7.0 milestone Mar 22, 2023
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.

Bug: can't remove administrator from user Bug: Original filename isn't saved in UTF-8
2 participants