Opened 8 days ago
Closed 7 days ago
#35995 closed Cleanup/optimization (fixed)
Suggestion: update reference to Yslow
| Reported by: | Justin Duke | Owned by: | amansharma612 |
|---|---|---|---|
| Component: | Documentation | Version: | 5.1 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | yes | UI/UX: | no |
Description
Hi there! I ended up on the Performance doc page (https://docs.djangoproject.com/en/5.1/topics/performance/#third-party-services) and noticed a link and recommendation to Yahoo's Yslow for general (non-Django) page speed debugging.
I think we should probably remove this reference; Yslow no longer is shipped as a chrome extension, and has not been updated in over a decade. I'd suggest (no affiliation besides happy user!) GTmetrix as a replacement.
Change History (5)
comment:1 by , 8 days ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:2 by , 8 days ago
| Easy pickings: | set |
|---|
comment:3 by , 7 days ago
| Has patch: | set |
|---|---|
| Owner: | set to |
| Status: | new → assigned |
Link to PR: https://github.com/django/django/pull/18918
comment:4 by , 7 days ago
| Triage Stage: | Accepted → Ready for checkin |
|---|
comment:5 by , 7 days ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
I forgot to include the ticket number in the commit message, fix for this ticket was merged in 6f38697f90a14f1450a71c1e40aea0f5df7dca86 and backported to 5.1 in 5f4252ecd6ce19cbf6d8bb9a3b4cca6a32eb40c2.
Thank you, Dustin. I agree with you. We should remove the two examples and avoid adding any specific tool names, as we risk recommending an outdated or unsupported tool in the near future.