Skip to content
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

Audit / remove old HTML #295

Closed
SamWhited opened this issue Nov 27, 2016 · 6 comments
Closed

Audit / remove old HTML #295

SamWhited opened this issue Nov 27, 2016 · 6 comments
Assignees

Comments

@SamWhited
Copy link
Member

There are several (S)HTML files scattered about the repo that can most likely be removed. The ones that provide redirects can be handled in the xsf/xmpp.org repo instead, or in the nginx config; either way they probably shouldn't be in the xeps repo.

@SamWhited SamWhited self-assigned this Dec 1, 2016
@stpeter
Copy link
Member

stpeter commented Dec 1, 2016

I see:

all.shtml
editor.shtml
index.shtml
ipr-policy.shtml
submit.shtml

I think all of these can be deleted.

And yes, we might need to put redirects in place in nginx.

@intosi
Copy link
Contributor

intosi commented Dec 1, 2016

I suspect you'll find that there are redirects in nginx for these already.

@SamWhited
Copy link
Member Author

SamWhited commented Dec 1, 2016

I'll open an issue against xsf/xmpp.org for creating redirects (or see if they already exist) and ask the iteam to make sure they don't need the shtml files for some reason.

@intosi
Copy link
Contributor

intosi commented Dec 1, 2016

@SamWhited really, no need:

rewrite ^/extensions/all.shtml http://xmpp.org/xmpp-protocols/xmpp-extensions/;
rewrite ^/extensions/editor.shtml http://xmpp.org/about-xmpp/xsf/xsf-people/#extensions;
rewrite ^/extensions/index.php http://xmpp.org/xmpp-protocols/xmpp-extensions/;
rewrite ^/extensions/index.shtml http://xmpp.org/xmpp-protocols/xmpp-extensions/;
rewrite ^/extensions/ipr-policy.shtml http://xmpp.org/about-xmpp/xsf/xsf-ipr-policy/;
[edit: submit.shtml is there, too]

They have been there for quite a while. I don't see any reason for keeping them.

@stpeter
Copy link
Member

stpeter commented Dec 1, 2016

Nuke 'em!

(Are we still allowed to say that?)

@SamWhited
Copy link
Member Author

Thanks all! I'll remove these "soon"

SamWhited added a commit to SamWhited/xeps that referenced this issue Dec 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants