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

ACF Country v2.0 - PHPv82 support #57

Merged
merged 1 commit into from
Nov 4, 2023
Merged

Conversation

joostdekeijzer
Copy link

Hi,

We're using your library for several projects. Since the official release is still v2.0 and we're upgrading to PHPv82 we encountered a deprecation notice about dynamic properties. This PR fixes that.

Prevents deprecation notice about dynamic properties.
@nlemoine nlemoine merged commit 2cd4528 into nlemoine:2.0 Nov 4, 2023
@nlemoine
Copy link
Owner

nlemoine commented Nov 4, 2023

Thanks @joostdekeijzer for your fix. This reminds me I should release version 3 officially, will do it soon.

@nlemoine
Copy link
Owner

nlemoine commented Nov 4, 2023

You'll find the new release details here: https://github.com/nlemoine/acf-country/releases/tag/2.1.0

@joostdekeijzer
Copy link
Author

Great & thanks!

Props for my college @Puntpixel who notified my of this issue and suggested the fix.

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.

2 participants