Danny van Kooten
Danny van Kooten
@Restingo Since the test is about making sure the Facade resolves correctly I would just pick another method on the class to test. For example: ```php self::assertEquals((new Countries())->hasCountryCode('NL'), CountriesFacade::hasCountryCode('NL')); ```
a4cdd08504eac2054f3fd8027c98fbd7fecbdf2d now retrieves the marketing permissions and stores them locally, but this only works if the list has at least 1 (pending or confirmed) subscriber. Next up is allowing to...
For anyone reading this, as of Mailchimp for WordPress version 4.9.2 (released earlier this week) you can now include a field named `MARKETING_PERMISSIONS` in your forms. The `value` attribute of...
Closing because of inactivity, but in case you're keen to get merged just let me know and we can pick up where we left.
Hello, I don't see this ending up in AltoRouter itself as the majority of users won't be using YAML but I am happy to list this as an available extension...
@marcusforsberg Terribly sorry for the late reply, but we'll be adding support for WooCommerce HPOS over the next few weeks.
@VincentLanglet Personally I am fine throwing a something like an `InvalidArgumentException` or `LogicException` (after a major version bump because of the BC break) in case the method is called with...
Hi @VincentLanglet, I'd like to go with throwing an `InvalidArgumentException()` when any of the `validateVatNumber*` methods are called with an unsupported country prefix. I'm fine with having an additional method...
Hello @trippo, I'm the maintainer yet I wholeheartedly agree with @VincentLanglet - unfortunately there are so many changes in this PR targeting several separate things that it's really hard for...
Hey @Stelikas, Thank you for the PR. Your first, that's pretty legendary! :tada: Did you run this code? Because unfortunately there should be some more required refactorings to account for...