-
Notifications
You must be signed in to change notification settings - Fork 123
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 way to delete saved REPLs #82
Comments
Being a noob myself, this are just my random thoughts and may not be accurate. Have you considered that this requested feature would not be desirable in the context of a shared environment such as Svelte.dev where the examples on the REPL need to be accessible for others to view, and accidental erasure could lead to losing examples required for issue reports? I don't yet know the structure of the REPL but, Would it be possible for you to simply delete the generated sub-directories on the command line to clean up on your forked version? |
I can share codepens and delete it at will, I can share jsfiddle and delete it at will. jsfiddle and codepen allow you to create anon repls which you can share and cannot be deleted. |
Very True, I suppose its just a matter of building the desired feature tied to a user ID and a no-delete flag. What I should have said is that with limited development resources, I guess it will depend on how many people see your feature request as being valuable enough before it will get attention. |
'Delete' could just remove them from your list but leave the repl untouched? Sounds like the problem is noise in your repl list rather than actually needing a repl deleted specifically. |
Another reason to be able to delete saved REPLs: I don't know if it's a bug with the REPL, GitHub authentication, Firefox, or some combination of the three, but when I work on a single REPL, login, then save it a single time, I get the following in my saved apps list. Note: I named the app "Recursive" because I was testing out |
Sorting alphabetically would be nice too. |
I agree, the option to delete (or at least remove them from my own list) would be fantastic. I've accidentally hit save on the That being said, maybe the Delete action should have some type of prompt in place: a kind of "ARE YOU SURE?" message to avoid accidental deletions as well. |
I think that with a fine warning message (ARE YOU SURE?) would be enough. If the owner of the repl decide to break existing references that's fine, as long as he is not doing it by accident. Moreover, the owner can alway edit the repl and leave it empty, so it's the same. |
...Or when the App.svelte is left empty, it should be removed. |
What I do is rename the title to empty. |
If I have a repl with private data in it, I want to be able to remove it from your server. This should be reason enough to allow people to delete their repls. If I add data, I should be able to remove it. |
If I understand correctly you can always edit your REPLS and replace them
with blank content, but the problem is that there will be an empty repl
left there forever...
…On Fri, Dec 4, 2020 at 8:42 PM Ben Woodward ***@***.***> wrote:
If I have a repl with private data in it, I want to be able to remove it
from your server. This should be reason enough to allow people to delete
their repls. If I add data, I should be able to remove it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sveltejs/svelte/issues/3457#issuecomment-739079208>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADVTF64K4XOKGDO5YHPPWLSTFXXJANCNFSM4IPEWJVQ>
.
|
If deleting a saved REPL is a problem, maybe the function... hide in the list https://svelte.dev/apps? Because it's really about getting this list right. It can work like this:
The code is still there, the REPL link is still working, Google is still indexing it. And that's all. Is this a good compromise? |
Please svelte team, We need this! And the autosave, Why don't you make a better REPL interface? |
I tried to tackle this a few weeks ago. It wasn't hard to support deleting repls from the database. The main issue was that there isn't a clear consensus how should we fix this. From the discussion, we know that:
Unresolved questions:
Before one actually sends a PR for this, we need these questions resolved. Otherwise, we'll only get them answered when the PR is reviewed (and probably cause wasted work). |
Just to give my two cents to the discussion, and specially because I think svelte repl is an esential tool to showcase svelte, teach how it works, report issues, and brag about it with the rest of the frameworks:
Unresolved questions: Taking the first option, I would mark the "visually deleted" items as
After you log in and go to your saved apps you will only see the active repls (those which are not Once you click Another option would be to make a
When you are browsing active repls, an Optionally each repl could also have a
Not sure about this one, I can imagine the following options
I think the most comfortable one for the user would be the third option, although I guess it would be a bit harder to implement |
FWIW - my $0.02 would be: keep it really simple at least at first:
At some point later we can think of more sophisticated stuff like archive/unarchive for example - if really needed. |
Note: We're currently migrating the site to https://github.com/sveltejs/sites through #78, which will be using a new backend at https://github.com/sveltejs/api.svelte.dev. If anyone wants to tackle this, I'd hold off until the PR is merged. |
This is simple but destructive, there is no walking that option back. I'd opt for just hiding it for now. To do this we need to store some hidden/ viewable metadata along with the gist. |
My main use case is that seeing all my old REPL's is a bunch of noise when I'm trying to find a specific one, so having the ability to hide/show them would be great, and maybe in the future being able to tag REPL's would be even better. And being able to create lists/groups/categories of REPL's would be even more better...er :-) I wouldn't necessarily take completely deleting REPL's off the table, but make it tough to do so, which would require a explicit and extensive action by the author to delete the REPL. |
Yes, perhaps it's better to go directly for the tagging option and handle deleted ones with an |
It's now possible to delete stuff via https://svelte.dev/apps |
Is your feature request related to a problem? Please describe.
Yes, being a noob I forked same stuff over and over trying to figure out how it works and my saved list quickly grew to same stuff that is no longer needed.
Describe the solution you'd like
Would like a delete button on each saved REPL so it doesn't pollute your server and keeps my list clean and clear.
Describe alternatives you've considered
Abandoning ship, formatting computer and starting all over.
How important is this feature to you?
Very, since I use REPL feature a lot for practice and learning and would like to keep it in orderly fashion.
Additional context
n/a
The text was updated successfully, but these errors were encountered: