Skip to content

Conversation

@shadinaif
Copy link
Collaborator

Description:

Our internal JSON structure for the theme designer

Related Issue:

Copy link
Collaborator

@OmarIthawi OmarIthawi left a comment

Choose a reason for hiding this comment

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

I'll continue from here.

@shadinaif shadinaif force-pushed the shadinaif/theme-designer-json branch 6 times, most recently from a1a348d to ec8aa08 Compare March 25, 2025 11:48
@shadinaif shadinaif force-pushed the shadinaif/theme-designer-json branch 2 times, most recently from 8f6c2c8 to bd79f12 Compare March 25, 2025 13:22
@shadinaif shadinaif force-pushed the shadinaif/theme-designer-json branch from bd79f12 to a36b8bd Compare March 26, 2025 14:35
}

root_settings = {
"logo_image_url": "url", #-- API key: logo_url
Copy link
Collaborator

Choose a reason for hiding this comment

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

right?

Suggested change
"logo_image_url": "url", #-- API key: logo_url
"LOGO_URL": "url", #-- API key: logo_url

Copy link
Collaborator Author

@shadinaif shadinaif Apr 3, 2025

Choose a reason for hiding this comment

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

LOGO_URL is the global fall-back. The one per tenant config is logo_image_url. Working fine in my local

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks @shadinaif!!

},
},

"footer": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

add dark/light style mode

Suggested change
"footer": {
"footer": {
"style": "dark",

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

will the style be applied to the footer only?

Copy link
Collaborator

Choose a reason for hiding this comment

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

will the style be applied to the footer only?

@shadinaif, I'm thinking of "template": "dark-v1" and "template": "light-v1". so we add more templates in the future as requested.

Copy link
Collaborator

@OmarIthawi OmarIthawi left a comment

Choose a reason for hiding this comment

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

Thanks @shadinaif and Mohamed! Looks great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants