This PR includes a bug where log files were flooded by element query executed before craft is fully initialised along with some code refactors to reduce some extra actions in order controller.
Related Issue(s):
456
Changes Made:
[List the changes made in this pull request to address the issue or implement the feature.]
Added:
A check before registering after plugin load events for if the craft is initialised.
Changed:
-
Deprecated:
-
Removed:
Clone order controller action that is to be done by using Save order action.
Fixed:
-
Security:
-
Testing:
[Describe the testing performed to ensure the changes are functioning as expected.]
Quality Assurance (QA):
[x] The code has been reviewed and approved by the QA team.
[x] The changes have been tested on different environments (e.g., staging, production).
[x] Integration tests have been performed to verify the interactions between components.
[x] Performance tests have been conducted to ensure the changes do not impact system performance.
[x] Any necessary database migrations or data transformations have been executed successfully.
[x] Accessibility requirements have been considered and tested (e.g., screen reader compatibility, keyboard navigation).
Resources:
Screenshots (if applicable):
[Include any relevant screenshots to visually demonstrate the changes.]
Wrapping up
Checklist:
[x] The code builds without any errors or warnings.
[x] The code follows the project's coding conventions and style guidelines.
[x] Unit tests have been added or updated to cover the changes made.
[x] The documentation has been updated to reflect the changes (if applicable).
[x] All new and existing tests pass successfully.
[x] The PR has been reviewed by at least one other team member.
Additional Notes:
[Include any additional notes, considerations, or context that may be relevant.]
Pull Request
Description:
This PR includes a bug where log files were flooded by element query executed before craft is fully initialised along with some code refactors to reduce some extra actions in order controller.
Related Issue(s):
456
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.]