Skip to content

allow uploading gltf/glb/fbx files #3494

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

Open
eyaler opened this issue May 24, 2025 · 5 comments · May be fixed by #3520
Open

allow uploading gltf/glb/fbx files #3494

eyaler opened this issue May 24, 2025 · 5 comments · May be fixed by #3520
Labels
Feature Request Proposal for adding a new functionality Ready for Work Issue is ready for development work

Comments

@eyaler
Copy link

eyaler commented May 24, 2025

Increasing Access

make it easier for students

Feature request details

glb/gltf/fbx files are blocked from upload. we currently do allow stl/mtl/obj, and moreover one can actually upload and use glb if giving them a fake extension.

@eyaler eyaler added Feature Request Proposal for adding a new functionality Awaiting Maintainer Approval Needs review from a maintainer before moving forward labels May 24, 2025
@raclim raclim added Ready for Work Issue is ready for development work and removed Awaiting Maintainer Approval Needs review from a maintainer before moving forward labels May 27, 2025
@webermayank
Copy link

@eyaler @raclim Hey , if this issue if ready for work, i would like to take this on, can i work on it ?

@G4URAV001
Copy link

G4URAV001 commented Jun 1, 2025

hey @eyaler @raclim , I have working on this and almost done with issue, just wanted to know that should I update the translation.json file for all the language for showing glb/gltf/fbx files in the warning message, mainly because this is not exactly translation just adding glb/gltf/fbx in all or just for English

Image

Updated error message -

Image

Current error message

Image
Image

OR should it be seperate PR/issue?

@ojuschugh1
Copy link

Hi @eyaler @raclim kindly please assign me this issue

ojuschugh1 added a commit to ojuschugh1/p5.js-web-editor that referenced this issue Jun 10, 2025
@ojuschugh1 ojuschugh1 linked a pull request Jun 10, 2025 that will close this issue
4 tasks
@ojuschugh1
Copy link

Hi @eyaler @raclim solved the issue in this PR #3520

@raclim raclim assigned ojuschugh1 and unassigned ojuschugh1 Jun 10, 2025
@raclim
Copy link
Collaborator

raclim commented Jun 10, 2025

Thanks so much @webermayank, @G4URAV001, and @ojuschugh1 for expressing interest in this issue! I'm sorry for the delay in getting back with a response.

I have working on this and almost done with issue, just wanted to know that should I update the translation.json file for all the language for showing glb/gltf/fbx files in the warning message, mainly because this is not exactly translation just adding glb/gltf/fbx in all or just for English

Thanks for linking screenshots @G4URAV001 and for pointing out questions about the translations! I think to help keep issues and concerns organized, it would be best to open a separate issue for these translations and have them incrementally handled there!

I'm also sorry for the delay in keeping with the recent assigned issue policy. Since @ojuschugh1 currently has an open PR, @G4URAV001 and @webermayank would you be open to maybe reviewing this with some of your suggestions? Let me know if you all have other ideas here as well!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Proposal for adding a new functionality Ready for Work Issue is ready for development work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants