Closed bhupeshappfoster closed 2 months ago
@sidedwards @shnsumit
The first step for craft 5 upgrade was to resolve issues in plugin code. So we did these changes to fix issues shown by phpstan and now the check is showing this
NOTE: I am doing extensive testing to confirm every thing is working as it was prior to these changes. Would be great to have a second pair of eyes too.
CC: @MerrittBoydston
Deployed at https://stage.acclarocraft.com/admin/dashboard
Pull Request
Description:
Craft suggested to use php stan before upgrading to craft 5 support and resolve any issues found.
Related Issue(s):
[Cite any related issues or feature requests, using the GitHub issue link.]
Changes Made:
[List the changes made in this pull request to address the issue or implement the feature.]
Added:
Changed:
-
Deprecated:
-
Removed:
-
Fixed:
Security:
-
Testing:
[Describe the testing performed to ensure the changes are functioning as expected.]
Quality Assurance (QA):
Resources:
Screenshots (if applicable): [Include any relevant screenshots to visually demonstrate the changes.]
Wrapping up
Checklist:
Additional Notes: [Include any additional notes, considerations, or context that may be relevant.]