-
Notifications
You must be signed in to change notification settings - Fork 12
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
Tool for adding new assets #155
Comments
fixed the sass issue and now got a local thing running @SpyTec what stuff would you want in the program, rn the only thing i am adding is just new assets, but i can see why it would be handy to have it also add new categories just in case. i'll work on it in the next few days, not an experienced dev in any way, but it should deffo be do-able! |
Hi @thijnmens! I did look into it a bit after I made that comment. It isn't possible to work with the GitHub API without a server since they do not support implicit OAuth flow. So that boils down to 3 possible solutions for a tool
Personally, I don't think doing any of these right now is the best way to reduce the contribution time. Instead I would love to refactor EDAssets completely to reduce the complexity in 3 ways
Sorry for the large amount of info! |
The project started in the early days of Angular when SPA was just becoming a thing. Even after refactoring to Vue the overall style of the project remained the same to keep backwards compatibility with the URLs |
I have already made a few websites with nextjs and this sounds like a fun project to work on tbh. Lots of stuff to learn, and it will definitely help with my education! I would love to work with you on this project ;), i'll deffo keep in mind all the suggestions you had, i especially like the idea of packaging all the files the user requested into a zip, kinda like a shopping cart xD |
I have created a fork of the current rep here, if you have any suggestions you can just comment on this issue or create a new one over at the fork, or if you want to work on it just create a pull request |
Creating a PR wouldn't hurt, it centralizes the feedback system |
True, welp you do you 👍, already made the base layout, tomorrow i am going to work on the shopping cart thingy and after that prob the database where the shopping cart can request the images from |
Oh I meant that you can create a PR towards this repo so the diff is shown |
ah ok, i created a draft PR for now, idk if this is what you meant, really not that experienced with github, but this should work |
Originally posted by @SpyTec in #134 (comment)
As mentioned here it would be nice to have a tool for uploading assets to make it a lot easier. Since i have nothing better to do and i want my assets uploaded i thought why not lol. Either way i am having issues setting up the project. npm gives loads of warnings and crashes on install node-sass, that i can fix, but it would be nice to know the node version used in this project so it doesn't give me all these errors when i try to start the project
The text was updated successfully, but these errors were encountered: