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

Listing-widget iFrame font #1368

Open
saumier opened this issue Sep 25, 2024 · 2 comments · Fixed by #1367
Open

Listing-widget iFrame font #1368

saumier opened this issue Sep 25, 2024 · 2 comments · Fixed by #1367
Assignees

Comments

@saumier
Copy link
Member

saumier commented Sep 25, 2024

As an admin in CMS, I want to share the widget by clicking the copy button without making any changes to the appearance.

However, when I look at the font attribute I get
&font=%5Bobject+Object%5D

Expected behviour is to get the default font
&font=Roboto

Note: This bug goes away once I start changing the appearance. The bug only happens if I don't change anything.

@saumier saumier added the bug Something isn't working label Sep 25, 2024
@saumier saumier changed the title Listing-widget iframe font Listing-widget iFrame font Sep 25, 2024
@SyamBabu-M
Copy link
Contributor

@saumier I have fixed this. You can test it here. Thanks

@SyamBabu-M SyamBabu-M assigned saumier and unassigned SyamBabu-M Sep 26, 2024
@saumier saumier transferred this issue from culturecreates/footlight-calendar-widget Sep 26, 2024
@saumier saumier assigned SyamBabu-M and unassigned saumier Sep 27, 2024
@saumier
Copy link
Member Author

saumier commented Sep 27, 2024

@SyamBabu-M Can you push this to Production? Can you also remind me what our dev process is for pushing to production? Thx.

@SyamBabu-M SyamBabu-M added To be deployed and removed bug Something isn't working labels Sep 27, 2024
@SyamBabu-M SyamBabu-M linked a pull request Sep 30, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants