Skip to content

Remove /projects/ #6221

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

Closed
stsewd opened this issue Sep 30, 2019 · 3 comments · Fixed by #6228
Closed

Remove /projects/ #6221

stsewd opened this issue Sep 30, 2019 · 3 comments · Fixed by #6228
Labels
Accepted Accepted issue on our roadmap Good First Issue Good for new contributors Improvement Minor improvement to code

Comments

@stsewd
Copy link
Member

stsewd commented Sep 30, 2019

All related code should be removed, that page isn't useful https://readthedocs.org/projects/

project_urls = [
url(r'^projects/', include('readthedocs.projects.urls.public')),
]

@stsewd stsewd added Good First Issue Good for new contributors Improvement Minor improvement to code Accepted Accepted issue on our roadmap labels Sep 30, 2019
@iambenzo
Copy link
Contributor

iambenzo commented Oct 1, 2019

I'd like to pick this one up please.

To be clear - are you wanting the removal of the publicly accessible pages (leaving behind the URLs for authenticated users)?

Or, do you want the entire route to be removed?

@saadmk11
Copy link
Member

saadmk11 commented Oct 1, 2019

I think here we want to remove the ProjectIndex view which serves https://readthedocs.org/projects/ and Everything related to it.

class ProjectIndex(ListView):

@iambenzo
Copy link
Contributor

iambenzo commented Oct 1, 2019

Okay, thanks for the clarification.

I will have a go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Good First Issue Good for new contributors Improvement Minor improvement to code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants