ScratchAddons icon indicating copy to clipboard operation
ScratchAddons copied to clipboard

New Addon: `project-folders`

Open Platformer444 opened this issue 2 years ago • 11 comments

Resolves #

Changes It adds project folders to organize your projects

Reason for changes It makes finding your projects easier

Tests Tested on Chrome 105 and it works fine

Platformer444 avatar Sep 19 '22 15:09 Platformer444

There is an API for my stuff, so project IDs should probably be fetched from https://scratch.mit.edu/site-api/projects/all/ instead.

Samq64 avatar Jan 31 '23 13:01 Samq64

There is an API for my stuff, so project IDs should probably be fetched from instead: https://scratch.mit.edu/site-api/projects/all/

Thank you for informing me about this but is there any API which fetches all the projects of the user and also includes their instructions and notes and credits?

Platformer444 avatar Jan 31 '23 16:01 Platformer444

I'm not sure, but I don't think so.

Samq64 avatar Jan 31 '23 16:01 Samq64

Unfortunately, I'm going to have to move this to the next milestone.

WorldLanguages avatar Feb 02 '23 23:02 WorldLanguages

I think I forgot to link it here, but https://github.com/Platformer444/ScratchAddons/pull/1

GrahamSH-LLK avatar Jul 24 '23 11:07 GrahamSH-LLK

I haven't considered that maybe we should try to reuse code from search-my-stuff in this addon, for filtering folders on the My Stuff page.

WorldLanguages avatar Jul 29 '23 22:07 WorldLanguages

I tried it myself, and I think this addon should not be accepted and the PR closed. Here's why:

  1. It's unneccessary. With studios which let you group your projects (albeit quite publicly) and the search-my-stuff addon you can already quite quickly find any project you're interested in.
  2. It's extremely slow to load. The loading time is just far too slow, and there's not much you can do about that if you really must have the data for the project folders being stored in project descriptions. Also every time you reload the page you have to wait for it to load again. It's so slow, that you could easily find your project just by scrolling in the time it takes to load.

Now problem #2 can (and should) be fixed by storing data in the addon itself. But that doesn't excuse the main problem that the idea is really un-neccessary given that we can search for projects with search-my-stuff. It's a big, complex addon that will waste resources, for little actual additional functionality.

Joeclinton1 avatar Jan 06 '24 22:01 Joeclinton1

I tried it myself, and I think this addon should not be accepted and the PR closed. Here's why:

  1. It's unneccessary. With studios which let you group your projects (albeit quite publicly) and the search-my-stuff addon you can already quite quickly find any project you're interested in.
  2. It's extremely slow to load. The loading time is just far too slow, and there's not much you can do about that if you really must have the data for the project folders being stored in project descriptions. Also every time you reload the page you have to wait for it to load again. It's so slow, that you could easily find your project just by scrolling in the time it takes to load.

Now problem #2 can (and should) be fixed by storing data in the addon itself. But that doesn't excuse the main problem that the idea is really un-neccessary given that we can search for projects with search-my-stuff. It's a big, complex addon that will waste resources, for little actual additional functionality.

https://github.com/Platformer444/ScratchAddons/pull/1 fixes the slow loading

GrahamSH-LLK avatar Jan 07 '24 16:01 GrahamSH-LLK

@GrahamSH-LLK Wow, that is alot faster. Well, then it comes down to the question of whether the added utility is worth mantaining this addon. I guess with so much work having already gone into it, it wouldn't hurt to see it come to frutition.

Since your change hasn't been merged yet and the original creator of the PR is inactive you may need to make a separate PR, with your version.

Joeclinton1 avatar Jan 07 '24 17:01 Joeclinton1

@Platformer444 Could you merge https://github.com/Platformer444/ScratchAddons/pull/1? We can take over from there if you're not interested in maintaining this.

Samq64 avatar Jan 19 '24 14:01 Samq64

Ping! There has been no activity for 7 days.

scratchaddons-bot[bot] avatar Jan 27 '24 00:01 scratchaddons-bot[bot]

closing due to inactive author. @Samq64 @GrahamSH-LLK feel free to take over

TheColaber avatar Jun 20 '24 04:06 TheColaber