-
Notifications
You must be signed in to change notification settings - Fork 48
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
Theme settings need cleanup #11
Milestone
Comments
Theme info proposal;
|
I'll add this bit in now. |
Do we remove the color options? I feel like changing colours won't be used by most merchants. The settings should all be merchant focused really. |
Yeah I think so. There should be minimal colour config available in the base framework. |
Cam
pushed a commit
that referenced
this issue
Aug 2, 2018
entry script should be theme.js instead of index.js
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The theme settings have reverted to an old version. We'll need to manually clean them up for the new methods.
The text was updated successfully, but these errors were encountered: