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

Update ACF Pro download endpoint #5

Merged
merged 2 commits into from
Mar 5, 2020
Merged

Conversation

nlemoine
Copy link
Contributor

@nlemoine nlemoine commented Mar 2, 2020

Hi @pivvenci

ACF download URL recently changed.

@nlemoine
Copy link
Contributor Author

nlemoine commented Mar 3, 2020

@schliflo I fixed the typo ;)

@Qrious
Copy link
Contributor

Qrious commented Mar 4, 2020

I'll update this, this also requires an update of our installer. I'll release the new version as well

@nlemoine
Copy link
Contributor Author

nlemoine commented Mar 4, 2020

I didn't notice download URL was also somewhere in installer. Thanks!

@Qrious
Copy link
Contributor

Qrious commented Mar 4, 2020

For now ACF's API is still backwards compatible :). I'm still giving this some thought, to make sure we have the best upgrade path. I think it is important that I update the installer in a backwards compatible manner.

I'll look further at this later today.

@Qrious
Copy link
Contributor

Qrious commented Mar 4, 2020

I've released a new version (2.3) of the installer. However I think we should only use the new download link for releases >= 5.8.8, and then also require ^2.3 of the installer for these newer versions. Otherwise current versions break with the old installers.

@Qrious
Copy link
Contributor

Qrious commented Mar 4, 2020

I've added these changes. What do you think, does this break compatibility with any of your projects (I don't think it should)?

@nlemoine
Copy link
Contributor Author

nlemoine commented Mar 4, 2020

For now ACF's API is still backwards compatible :)

I just checked, it is ok now but I actually made this PR because I had a 404 on the old download link a few days ago.

What do you think, does this break compatibility with any of your projects (I don't think it should)?

I think you shouldn't bother to target different URLs, the new endpoint works fine with every ACF version ;)

@Qrious Qrious merged commit d632846 into pivvenit:master Mar 5, 2020
@Qrious
Copy link
Contributor

Qrious commented Mar 5, 2020

That does however break backwards compatibility with people that have the old versions in their lock file, together with the old installers, so i've decided to support the new URL's for the latest release and upwards (as does ACF-PRO when used without composer).

@nlemoine
Copy link
Contributor Author

nlemoine commented Mar 5, 2020

I didn't catch the installer issue. Great job. Thanks for merging this!

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