Skip to content
This repository has been archived by the owner on Jul 28, 2023. It is now read-only.

⚛️ Restructure the project folder and files to make it ready for the Directives Hydration #65

Conversation

luisherranz
Copy link
Member

@luisherranz luisherranz commented Sep 6, 2022

Co-authored with @DAreRodz.

These are the things we did here:

  • Clean a bit the code related to the Custom Elements Hydration
  • Create a separate config for Webpack for the hydration code and the View components
  • Enqueue the hydration and View components
  • Switch to Preact (compat) on that new Webpack config
  • Remove the shared code from the Editor to avoid any @wordpress import
    In doing so, we broke the code of the Editor, but we'll fix that in an upcoming PR

The folder/files should be ready now to add the first implementation of the Directives Hydration.

luisherranz and others added 5 commits September 6, 2022 16:15
Co-authored-by: David Arenas <david.arenas@automattic.com>
Co-authored-by: David Arenas <david.arenas@automattic.com>
Co-authored-by: David Arenas <david.arenas@automattic.com>
Co-authored-by: David Arenas <david.arenas@automattic.com>
Co-authored-by: David Arenas <david.arenas@automattic.com>
@luisherranz luisherranz self-assigned this Sep 6, 2022
@luisherranz luisherranz changed the base branch from main-island-hydration to main-full-vdom-hydration September 6, 2022 16:19
Copy link
Collaborator

@DAreRodz DAreRodz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! ✅

@DAreRodz DAreRodz merged commit b39e7d6 into main-full-vdom-hydration Sep 7, 2022
@DAreRodz DAreRodz deleted the full-vdom/structure-refactoring-and-initial-implementation branch September 7, 2022 16:02
@luisherranz luisherranz changed the title ⚛️ Restructure the project folder and files to make it ready for the Full vDOM hydration ⚛️ Restructure the project folder and files to make it ready for the Directives Hydration Jan 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants