This repository was archived by the owner on Apr 18, 2024. It is now read-only.

WIP rewrite for Craft 4 - #69

Open

ostark wants to merge 3 commits into

ostark commented Jul 12, 2022•
edited
Loading

Owner

Discussions #68 & #65
Issue #67

So far no significant code changes, just structural.

Prepare for tests with pest
Use Handler classes instead of closures
Use DI Container instead of Service Locator (not everywhere so far)
Move event listener setup to the main Plugin class (remove EventRegistrar)
Refactor drivers (using https://github.com/Sammyjo20/Saloon)

Oliver Stark added 3 commits

July 12, 2022 12:53

Test setup and composer version bump

7014fdf

Namespace changes

7b67f49

Refactor huge EventRegistrar to multiple invokeable handler classes

c37b345

ostark changed the title ~~Craft 4 rewrite~~WIP rewrite for Craft 4

mattbloomfield commented May 30, 2023

Hey @ostark any progress here?

Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.