Skip to content
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
masterfrom
v2-dev
Open

WIP rewrite for Craft 4#69
ostark wants to merge 3 commits into
masterfrom
v2-dev

Conversation

@ostark

@ostark ostark commented Jul 12, 2022 •
edited
Loading

Copy link
Copy Markdown
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)
@ostark ostark changed the title Craft 4 rewrite Jul 12, 2022
@mattbloomfield

mattbloomfield commented May 30, 2023

Copy link
Copy Markdown

Hey @ostark any progress here?
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants

@ostark @mattbloomfield