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

Berry widget demo #4946

Draft
wants to merge 37 commits into
base: main
Choose a base branch
from
Draft

Berry widget demo #4946

wants to merge 37 commits into from

Conversation

chvik
Copy link
Collaborator

@chvik chvik commented Jan 9, 2025

Purpose of this pull request

This change integrates the Berry widget based on #4748 with the following updates:

  • Adds the Berry widget inline on the homepage and as a popup elsewhere.
  • Synchronizes the Berry widget's color mode with the site's color mode.
  • Increases the height of the inline widget.
  • Removes the resize button from the inline widget.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

Copy link

cla-bot bot commented Jan 9, 2025

Thank you for your contribution! As this is an open source project, we require contributors to sign our Contributor License Agreement and do not have yours on file. To proceed with your PR, please sign your name here and we will add you to our approved list of contributors.

@kimsauce kimsauce changed the base branch from main to berry January 9, 2025 21:20
kimsauce
kimsauce previously approved these changes Jan 9, 2025
@kimsauce kimsauce self-assigned this Jan 9, 2025
@kimsauce kimsauce added the do-not-merge Pull requests that should not be merged label Jan 9, 2025
@kimsauce kimsauce self-requested a review January 9, 2025 22:01
@kimsauce kimsauce marked this pull request as draft January 9, 2025 22:01
@kimsauce kimsauce changed the base branch from berry to main January 9, 2025 22:01
@kimsauce kimsauce dismissed their stale review January 9, 2025 22:02

outdated

@kimsauce

This comment was marked as resolved.

This comment was marked as resolved.

@chvik chvik temporarily deployed to review/4946/merge January 10, 2025 11:06 — with GitHub Actions Inactive
@chvik
Copy link
Collaborator Author

chvik commented Jan 10, 2025

Hi @chvik, thanks so much for all your hard work on this! I just ran into one snag: everything worked locally on your branch, but after syncing it with main, the inline widget broke (screenshot below). We’ve made some back-end Docusaurus/package updates over the past week that likely aren’t reflected in your build yet. Would you be able to pull in main and try syncing again? Sorry for the extra trouble—I really appreciate your help!

Hi @kimsauce,

Coincidentally, we introduced an extra firewall protection in front of our application website yesterday, which I believe caused the issue you experienced. Since then, we’ve relaxed the rules, and the issue should now be resolved. I’ve merged main into the branch, and everything is working fine on my end.

@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Jan 10, 2025
@kimsauce kimsauce temporarily deployed to review/4946/merge January 10, 2025 23:31 — with GitHub Actions Inactive
@kimsauce kimsauce mentioned this pull request Jan 13, 2025
4 tasks
@kimsauce kimsauce added the doc:site tech Site/tool maintenance, dependencies, updates label Jan 13, 2025
@kimsauce kimsauce changed the title Integration of Berry Widget Berry widget demo Jan 15, 2025
@kimsauce kimsauce temporarily deployed to review/4946/merge January 15, 2025 05:23 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge January 15, 2025 05:55 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed Contributor approved, listed in .clabot file do-not-merge Pull requests that should not be merged doc:site tech Site/tool maintenance, dependencies, updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants