Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generally improve Automations user experience and code #99

Open
MichelleBlanchette opened this issue Oct 16, 2022 · 2 comments
Open

Generally improve Automations user experience and code #99

MichelleBlanchette opened this issue Oct 16, 2022 · 2 comments
Labels
cleanup Improves code organization and usability design Changes related to UX and styles performance Changes related to performance optimization

Comments

@MichelleBlanchette
Copy link
Collaborator

Sadly, this will require a migration process so that Automation data is maintained between versions. It's certainly worth it to standardize and simplify the data storage and interfacing for Automations.

@MichelleBlanchette MichelleBlanchette added documentation Related to documentation enhancement Existing feature enhancements cleanup Improves code organization and usability labels Oct 16, 2022
@MichelleBlanchette
Copy link
Collaborator Author

What is the goal of this? Could this instead be a problem with the database CRUD functions being excessively complex? Maybe the code could be simplified and some performance benchmarks should be recorded. Also, the frontend could maybe be changed to support a simpler backend architecture.

@MichelleBlanchette MichelleBlanchette added performance Changes related to performance optimization and removed enhancement Existing feature enhancements labels Apr 12, 2023
@MichelleBlanchette MichelleBlanchette changed the title Migrate Automations from custom database tables to post types Generally improve Automations user experience Apr 12, 2023
@MichelleBlanchette MichelleBlanchette changed the title Generally improve Automations user experience Generally improve Automations user experience and code Apr 12, 2023
@MichelleBlanchette MichelleBlanchette added design Changes related to UX and styles and removed documentation Related to documentation labels Apr 12, 2023
@MichelleBlanchette
Copy link
Collaborator Author

Note that there's several database queries executed on every page load related to retrieving Automations records. It's probably due to trying to hook them into the WordPress event system, but it still seems inefficient. Shouldn't you only need their hook name to register their behavior, and then just retrieve their full data when the hook actually executes? Additionally, the WordPress caching system can probably help further optimize this. I was certainly thinking of using it to optimize the main, Automations listing page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Improves code organization and usability design Changes related to UX and styles performance Changes related to performance optimization
Projects
None yet
Development

No branches or pull requests

1 participant