Ottawa
Account created on 4 March 2005, over 20 years ago
#

Merge Requests

More

Recent comments

🇨🇦Canada danrod Ottawa

I'll merge it to the 2.0.x, if anyone has any suggestions or things to add, please let me know and I'll re-open the issue.

🇨🇦Canada danrod Ottawa

Hi @d.fisher I think you can go ahead.

Don't forget to git rebase & apply the patch just in case.

🇨🇦Canada danrod Ottawa

Thanks again, I'll merge to the 2.0.x branch.

🇨🇦Canada danrod Ottawa

Thanks @ankitv18 for this, I see a deprecation warning, I'll look into this, but it looks ok.

The method $this->assertSession()->statusCodeEquals(200) is deprecated and some cases it was failing on me, so I had to delete it.

Using PHPUnit 11.5.34 and PHP 8.4.10 for the tests.

🇨🇦Canada danrod Ottawa

Thanks a bunch @ankitv18 !!

I admit that the Tugboat settings are new to me and I haven't had the time to read through the documentation to work on this, thanks a lot again.

Giving the credits and merging.

🇨🇦Canada danrod Ottawa

Hi @ankitv18 , thanks for letting me know. I'll fix it now (for both branches).

🇨🇦Canada danrod Ottawa

Thanks @ankitv18 , I fixed the link. Moving this to "Fixed"

🇨🇦Canada danrod Ottawa

Thanks a lot @ankitv18, fixed and deleted some redundant authors:

🇨🇦Canada danrod Ottawa

Thanks a lot for fixing the ESLINT and STYLEINT issues, @ankitv18

🇨🇦Canada danrod Ottawa

I think it looks better that way, a review would be nice.

Moving this to "Needs Review".

🇨🇦Canada danrod Ottawa

danrod changed the visibility of the branch 3542766-update-readme.md-to-8x to active.

🇨🇦Canada danrod Ottawa

danrod changed the visibility of the branch 3542766-update-readme.md-to-8x to hidden.

🇨🇦Canada danrod Ottawa

Fixed the PHPCS and PHPSTAN errors, kindly review.

🇨🇦Canada danrod Ottawa

I'll look into the PHPCS / PHSTAN errors

🇨🇦Canada danrod Ottawa

Tested and worked great, merging this to the 3.0.x branch.

🇨🇦Canada danrod Ottawa

danrod made their first commit to this issue’s fork.

🇨🇦Canada danrod Ottawa

No problem, you will have to use the dev release for now, until I fix some critical issues and then I'll create a tagged release.

🇨🇦Canada danrod Ottawa

Hello,

I created a MR against the 2.0.x branch to fix these PHP 8.4 deprecation issues, I hope it's the correct branch, if not, I can do the same for the 8.x-1.x branch.

Kindly review, thanks.

🇨🇦Canada danrod Ottawa

danrod changed the visibility of the branch 3542886-php-8.4-support to hidden.

🇨🇦Canada danrod Ottawa

Thanks @anweshas you are right, the cart is fine as it is now and will be up to the frontend developer to make the changes as desired.

I'll move it as "Closed (works as designed)"

Thanks for your input.

🇨🇦Canada danrod Ottawa

I'll move it to "Needs work" for now. I'll probably continue working with it tomorrow.

🇨🇦Canada danrod Ottawa

The README.md file looks ok now, probably needs to be updated to the other branches as well. I'll set it to "Needs Review" for now.

🇨🇦Canada danrod Ottawa

Changed the README.md file and looks better now, I'll appreciate a review (I'll give a credit).

Moving this to "Needs Review".

🇨🇦Canada danrod Ottawa

It looks better now, I'll move it to "Needs Review", hopefully someone will look on that.

🇨🇦Canada danrod Ottawa

Looks much better now, an extra look would be nice. Setting this to "Needs Review".

🇨🇦Canada danrod Ottawa

I think It looks ok now, a review will be nice (will give a credit !!).

Moved to "Needs Review".

🇨🇦Canada danrod Ottawa

Thanks @neerajsingh, and I agree with removing the extra return before the execution of batch_set.

Merging this to the 3.0.x branch.

Thanks !

🇨🇦Canada danrod Ottawa

Looks ok to me @neerajsingh and thanks for fixing the PHPCS issues.

I'll merge this to the 3.0.x branch.

🇨🇦Canada danrod Ottawa

danrod created an issue.

🇨🇦Canada danrod Ottawa

Thanks @neerajsingh

Production build 0.71.5 2024