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

Support app drop in theme app extensions #566

Merged
merged 1 commit into from
Mar 25, 2022
Merged

Conversation

veken1199
Copy link
Contributor

@veken1199 veken1199 commented Mar 24, 2022

Context

We would like to support App Liquid drop in theme app extension Liquid code such as app blocks and embeds.

How did we implement this change?

We updated the initializer of UndefinedObject checker to take a new arg called config_type which is set to :default as the default value. We updated then theme app extension config file to pass the config_type of UndefinedObject checker as :theme_app_extension. Base on the config_type` we will be able to allow and reject more liquid objects in theme app extension liquid code.

@veken1199 veken1199 requested review from catlee and charlespwd March 24, 2022 17:57
Copy link
Contributor

@charlespwd charlespwd left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@veken1199 veken1199 self-assigned this Mar 25, 2022
@edo888
Copy link

edo888 commented Dec 29, 2022

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants