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

Moved templates from old folder location to new #6073

Closed
wants to merge 1 commit into from

Conversation

pbking
Copy link
Contributor

@pbking pbking commented Jun 6, 2022

This changes the templates (and template parts) folder location from the old location to the new. It also updates the "pattern registration" that registers Blockbase's template parts as patterns.

Intended for Blockbase 3

@pbking pbking added this to the Blockbase 3.0 milestone Jun 6, 2022
@pbking pbking requested a review from a team June 6, 2022 14:55
Copy link
Contributor

@MaggieCabrera MaggieCabrera left a comment

Choose a reason for hiding this comment

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

The changes LGTM but I wonder if we should divide this into two PRs to avoid breakage during deploy. I've had that happened to me when renaming an included file, don't know if this would be the case for dotcom, you may get alerts even if for just a few minutes.

@mikachan mikachan linked an issue Jun 9, 2022 that may be closed by this pull request
pbking added a commit that referenced this pull request Jul 20, 2022
Refactor/blockbase color admin (#6043)
Moved templates from old folder location to new (#6073)
Blockbase: Implement the Button elements API (#6041)
Blockbase: Implement Comment Block and removed CSS (#6080)
Fix/migrate blockbase font self hosted (#6123)
Blockbase children: update comments block (#6153)
Blockbase: Changed the trigger to render social icons (#6079)
Blockbase: move button padding styles from ponyfill to theme.json (#5901)

Co-authored-by: Grant Kinney <[email protected]>
Co-authored-by: Jeremy Yip <[email protected]>
Co-authored-by: MaggieCabrera <[email protected]>
Co-authored-by: madhusudhand <[email protected]>
@pbking pbking mentioned this pull request Jul 20, 2022
@pbking
Copy link
Contributor Author

pbking commented Jul 20, 2022

Closing: Work merged in #6167

@pbking pbking closed this Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Blockbase: Update name of template directories
2 participants