-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat!: Cleanup Failing Patches #153
Conversation
TODO: Check all respective landed @podarok
Refer to this link for build results (access rights to CI server needed): Check YMCA Website Services Installation Wizard at http://install.openy.cibox.tools/build4577/install.php |
YMCA Website Services Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build4577/ |
Open Y Complete profile installed at http://profile.openy.cibox.tools/build4577/ |
no more errors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm ok with the deleting old and outdated patches that are already in code.
We just have to more carefully do regression testing when its planned
I'll make it after 10.3.2 release @anpolimus |
TODO: Check all respective landed @podarok
Original Issue, this PR is going to fix: REPLACE WITH A LINK TO ISSUE ( publicly available )
Make sure these boxes are checked before asking for review of your pull request - thank you!
If there is a new feature or this is a bug fix - use
main
branch. We'll tag for release if the bug is critical ASAP or tag for release next bug fix release until the critical issue arrives.Steps for review
General checks
Thank you for your contribution!