Make delete project work without javascript

Most of our major functions work without javascript, but not delete.

Someone suggested: As it stands today, I see two main methods for deleting the project (I didn't check to see if the API supports it; I'm referring to the UI): through the project settings, which has you verify the project name by typing it, and through the main project page, which has you click "OK" (which makes me uncomfortable).

The only safe option I see is to bring the user to another page to verify their action, which can be done in the same manner as it is today: enter the name of the project and submit.

I'm not suggesting removing the existing implementation---just falling back.

subscribe

I made this suggestion.

The motivation is two-fold:

1. Users may choose not to run JavaScript, may be in an environment that does not support JavaScript (such as a text-mode browser), or may be automating certain actions using software like cURL; and

2. To ensure that JavaScript code that is not LibreJS-compliant will not inhibit "essential" site functionality.

I intend to submit a patch for the latter at some point, but the former is still a concern. I unfortunately do not have the time at the moment to research how to write a patch for this change.

Ideally:

• Existing functionality will be retained, as it provides a better user experience when JavaScript is enabled; and
• When JavaScript is not enabled, the user will be directed to another page with a normal HTML form (no JavaScript) that will prompt her to enter the name of the project for verification.

---

This is what I considered to be "essential" functionality during my evaluation:

• Browsing projects and code;
• Creating a new project;
• Delete a project;
• Change project settings;
• Update profile;
• Add account e-mail;
• Add SSH key;
• Add issue; and
• Create merge a request.

All worked without JavaScript with the exception of deleting a project. I applaud GitLab for this.

@rspeicher Would you be so kind to add "When JavaScript is not enabled, the user will be directed to another page with a normal HTML form (no JavaScript) that will prompt her to enter the name of the project for verification." to GitLab?

@dzaporozhets I hope you're OK with ^^^

@sytses I can look into it. Do we have a timeframe?

@rspeicher Not sure, @mikegerwitz has a deadline and it still needs to be packaged and deployed before that. 8.0.0rc1 would be nice if that is not a problem for you

@dzaporozhets Any comments on what @sytses mentioned here?

mentioned in merge request !1294 (closed)

@rspeicher I left comment in MR

mentioned in merge request !1294 (closed)

mentioned in merge request !1294 (closed)

mentioned in merge request !1527 (merged)

Status changed to closed by commit 75bb1087

mentioned in commit 75bb1087

mentioned in commit ac378e5b

mentioned in issue #15678 (moved)

mentioned in merge request !14785