Allow only post requests for delete views #6242
Conversation
| {% blocktrans trimmed %} | ||
| This version is inactive but its documentation is still available online. | ||
| You can <a href="{{ version_delete_url }}">delete this version's documentation</a> if you want to remove it completely. | ||
| You can <a href="#" onclick="document.forms['version_delete_html'].submit();">delete this version's documentation</a> if you want to remove it completely. |
There was a problem hiding this comment.
I'm using this here because using a button here doesn't look good also it breaks lot of CSS styles. Would love to get some Suggestions. :)
There was a problem hiding this comment.
I don't have a suggestion for this, and also this is something that users shouldn't hit that much.
There was a problem hiding this comment.
I think it makes sense for now.
I'd appreciate if you add this comment into the template itself (using {# #}) so we know why this done in this way in the future. Thanks!
|
Thanks, can you attach some screenshots of the final result? |
| {% blocktrans trimmed %} | ||
| This version is inactive but its documentation is still available online. | ||
| You can <a href="{{ version_delete_url }}">delete this version's documentation</a> if you want to remove it completely. | ||
| You can <a href="#" onclick="document.forms['version_delete_html'].submit();">delete this version's documentation</a> if you want to remove it completely. |
There was a problem hiding this comment.
I don't have a suggestion for this, and also this is something that users shouldn't hit that much.
| {% blocktrans trimmed %} | ||
| This version is inactive but its documentation is still available online. | ||
| You can <a href="{{ version_delete_url }}">delete this version's documentation</a> if you want to remove it completely. | ||
| You can <a href="#" onclick="document.forms['version_delete_html'].submit();">delete this version's documentation</a> if you want to remove it completely. |
There was a problem hiding this comment.
I think it makes sense for now.
I'd appreciate if you add this comment into the template itself (using {# #}) so we know why this done in this way in the future. Thanks!
closes: #6189