Now the tests are as green as they can be
Can't decide yet, my team will attempt to use it on prod and if then we'll see how it goes :)
How about the https://github.com/lcache/lcache, is there anyone that would take care of PHP8 compatibility?
In the MR I:
- implemented the new solution for authentication
- replaced the deprecated endpoint URLs
- updated README with more specific description of what settings are required
I also tested 2 of the services: DDS and ONIX, as these were the only services I had credentials to.
I'd need someone to review the changes.
juliusz_cheddar β created an issue.
The dependencies are fixed, so I'd say there's nothing more to do within the scope of this issue. Closing.
This patch consists of implementation of the described features
juliusz_cheddar β created an issue.
Agreed. My mistake
Just a small update:
Current state is that the unit tests fail in the CI, but mostly (that's a keyword, explained below) succeeds locally if you have the module enabled before running tests. If you won't enable the module, the lcache/lcache library won't find the lcache_events table which leads to an error.
The tests mostly succeeds, because there are 2 tests that fail because the timestamp of cached item is 1ms different from request time it's compared to. It sometimes succeeds, but sometimes fails.
juliusz_cheddar β created an issue.
juliusz_cheddar β created an issue.
Note from one of the mainterners of bokbasen/php-sdk-auth and bokbasen/php-api-client:
As you've notices, these packages hasn't been given much love in quite some time. In the not-so-distant-future we will move to auth0. But I think it should be somewhat smooth migrating to auth0/auth0-php for you.
Update 2:
Now we're blocked by https://github.com/Bokbasen/php-sdk-auth/pull/3
Update:
Now we're blocked by https://github.com/Bokbasen/php-api-client/pull/3
Not ready yet. The composer.json requires php-http/guzzle7-adapter (for compatibility with D10 core requirements), but zaporylie/vipps:^2 requires php-http/guzzle6-adapter
I will :)
Right now we're blocked by https://github.com/Bokbasen/php-sdk-auth/pull/2
juliusz_cheddar β created an issue.
I just fixed as much as I could. Fixed the tests and a lot of codestyle errors.
juliusz_cheddar β made their first commit to this issueβs fork.
Fixing the Unit and Kernel tests
Looks and works fine
juliusz_cheddar β created an issue.
juliusz_cheddar β created an issue.
Fixed the small coding standards issue.
Sorry for the #17, there was a misclick..
Here's the correct file uploaded
Attempt to fix the Unit tests.
Fixed coding stardards
I created a patch that makes the CommerceAjaxCartMessageSubscriber compatible with commerce version 2.36 and lower.
MR is also created. Please review
juliusz_cheddar β made their first commit to this issueβs fork.