Making WordPress.org

Opened 6 months ago

Closed 6 months ago

Last modified 6 months ago

#7257 closed defect (bug) (fixed)

Credits-API: Update some outdated URLs for external libraries

Reported by: michi91's profile Michi91 Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: Priority: normal
Component: API Keywords: has-patch
Cc:

Description (last modified by SergeyBiryukov)

Bug was reported here: #WP58796 by sumitsingh

The URL's for imgAreaSelect, PclZip and jQuery UI Touch Punch are broken (404).

Broken Links:

Alternative Links:

Related tickets in the past:

If you take a look at changesets in the past:

you can see, that updates where handled differently: Sometimes all releases where updated, sometimes some releases and somtimes only the latest.

desrosj personally thinks we should either:

  • Update 6.3 only
  • Update 6.0, 6.1, 6.2, and 6.3 (these 4 versions account for 75% of all WordPress sites)

Change History (7)

This ticket was mentioned in PR #176 on WordPress/wordpress.org by @Michi91.


6 months ago
#1

  • Keywords has-patch added; needs-patch removed

#2 @SergeyBiryukov
6 months ago

  • Description modified (diff)

#3 @SergeyBiryukov
6 months ago

  • Description modified (diff)

#4 @SergeyBiryukov
6 months ago

  • Owner set to SergeyBiryukov
  • Status changed from new to accepted

#5 @SergeyBiryukov
6 months ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 12878:

Credits API: Update URLs for external libraries.

Includes replacing a few broken links and using HTTPS for all URLs.

Follow-up to [160], [1048], [3917], [9194].

Props Michi91, sumitsingh, costdev, oglekler, desrosj, SergeyBiryukov.
Fixes #7257. See #WP58796.

#6 in reply to: ↑ description @SergeyBiryukov
6 months ago

Hi there, welcome to WordPress Meta Trac! Thanks for the ticket.

desrosj personally thinks we should either:

  • Update 6.3 only
  • Update 6.0, 6.1, 6.2, and 6.3 (these 4 versions account for 75% of all WordPress sites)

I think the former would be enough for now.

Found a few more links to update, [12878] should cover them all.

@SergeyBiryukov commented on PR #176:


6 months ago
#7

Thanks for the PR! Merged in r12878.

Note: See TracTickets for help on using tickets.