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

A super cool zoom machine #686

Merged
merged 24 commits into from
Feb 13, 2025
Merged

A super cool zoom machine #686

merged 24 commits into from
Feb 13, 2025

Conversation

TacticalTechJay
Copy link
Collaborator

  • Not mobile compatible though :(
  • fix: made a weird 404 for reply.notFound() whenever yarn deving cuz the original took too long to load :( (I don't think it even loads in the first place)
  • fix: check password before doing a preFile
  • fix: raw route has the password checking now, no more need for /api/auth/file! (and I only just renamed it ._.)
  • fix: /uploads/dashboard will NOT redirect to dashboard (url ditto)
  • notFix: force delete with local and s3 to avoid errors. Double check if this is necessary.
  • antiFix: No more rawdogging your datasources if they don't exist in the db, upload your files properly! >:(
    • Or you'll get a 404

@TacticalTechJay TacticalTechJay requested a review from diced February 7, 2025 03:26
@TacticalTechJay TacticalTechJay marked this pull request as draft February 9, 2025 20:45
@TacticalTechJay TacticalTechJay marked this pull request as ready for review February 12, 2025 05:11
@TacticalTechJay
Copy link
Collaborator Author

I've got embeds fixed up, it should work. I've added an /oembed api route, but that's only for discord so no documentation should be made for it

@diced diced merged commit 6646c1e into diced:trunk Feb 13, 2025
1 check passed
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