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

Remove deprecated Django-powered Sites link #1471

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

WajahatKanju
Copy link
Contributor

Summary

This pull request addresses the recent shutdown of djangosites.org by removing the now obsolete "Django-powered Sites" link from the Community page. The website, as mentioned in the "Farewell, Djangosites" article, is no longer available.
Resolves issue #1470

Changes Made

  • Removed the "Django-powered Sites" link from the Community page.
  • Considered community suggestions from the forum thread, confirming the consensus for link removal.

Context

The djangosites.org website has been shut down, and keeping the link in the Django documentation is misleading. This change improves the user experience by removing deprecated and non-functional links, aligning with the community's consensus.

Related Forum Thread

Link to the Forum Thread

Notes for Reviewers

Your feedback on this change is highly appreciated. Please let me know if any further adjustments are needed.

@rasulkireev
Copy link

@WajahatKanju

Can I ask you to update the PR to point to builtwithdjango.com instead of removing completely?
https://forum.djangoproject.com/t/replace-django-powered-sites-link-in-community-sections/27562

If you don't have the time, let me know, I ca do this myself. Thanks a ton in advance.

@WajahatKanju WajahatKanju force-pushed the fix/djangosites-link-removal branch from 9ea8fb7 to d155b17 Compare February 2, 2024 15:28
@WajahatKanju
Copy link
Contributor Author

@rasulkireev
I've updated it to point to builtwithdjango.com instead of removing the link completely. Please review the changes, and if everything looks good. If you have any further feedback or if there's anything else I can help with, please let me know.

Thanks!

@@ -44,8 +44,7 @@ <h2>More Links</h2>
<dl class="list-links">
<dt><a href="https://www.djangopackages.com/" rel="nofollow">Django Packages</a></dt>
<dd>Find third-party packages to supercharge your project</dd>
<dt><a href="https://www.djangosites.org/" rel="nofollow">Django-powered Sites</a></dt>
<dd>Add your site to the list</dd>
<dt><a href="https://builtwithdjango.com/" rel="nofollow">Django-powered Sites</a></dt>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also, should probably point to https://builtwithdjango.com/projects/ so that people see the projects right away without having to navigate to it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@WajahatKanju WajahatKanju force-pushed the fix/djangosites-link-removal branch from 7990b6b to a4908ee Compare February 2, 2024 15:34
@WajahatKanju WajahatKanju force-pushed the fix/djangosites-link-removal branch from a4908ee to 522f014 Compare February 2, 2024 15:35
@WajahatKanju
Copy link
Contributor Author

Done

@camilonova camilonova merged commit 8845158 into django:main Feb 9, 2024
2 checks passed
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

Successfully merging this pull request may close these issues.

3 participants