Skip to content

DOCS-543 - Berry widget demo #4946

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 62 commits into
base: main
Choose a base branch
from
Draft

DOCS-543 - Berry widget demo #4946

wants to merge 62 commits into from

Conversation

chvik
Copy link
Collaborator

@chvik chvik commented Jan 9, 2025 •
edited by kimsauce
Loading

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)

https://sumologic.atlassian.net/browse/DOCS-543

kimsauce and others added 29 commits November 10, 2024 22:36
@kimsauce
Berry widget demo env
15afb7c
@kimsauce
staging
e2c4b56
@kimsauce
Update .github/workflows/pr.yml
a075f6a
@kimsauce
edits
9248750
@kimsauce
edits
fc72909
@kimsauce
fix
5f5b918
@kimsauce
fix link
b7afc94
@kimsauce
Merge branch 'main' into berry
64fce9c
@kimsauce
Merge branch 'main' into berry
fba74a2
@kimsauce
Add suggested question prompts
3446d32
@kimsauce
Merge branch 'berry' of github.com:SumoLogic/sumologic-documentation …
2dc71f8 
…into berry
@kimsauce
Merge branch 'main' into berry
8104cd7
@kimsauce
Merge branch 'main' into berry
98f1b35
@kimsauce
undo
0d38907
@kimsauce
Update src/pages/index.tsx
40cd418
@kimsauce
undo
c43d3bd
@kimsauce
Merge branch 'berry' of github.com:SumoLogic/sumologic-documentation …
07ba48e 
…into berry
@kimsauce
Update src/css/sumo.scss
b21627f
@kimsauce
Merge branch 'main' into berry
0906449
@kimsauce
Merge branch 'main' into berry
2821875
@kimsauce
redesign
f22f120
@kimsauce
Merge branch 'main' into berry
b465d27
@kimsauce
css fixes, add more Qs
4d94f3c
@kimsauce
Merge branch 'main' into berry
e11638c
@kimsauce
Merge branch 'main' into berry
adca3ec
@kimsauce
Merge branch 'main' into berry
fb8772c
@chvik
install Berry popup on every page except on the home page
3dfb3fd
@chvik
increase height of inline widget
dd80617
@chvik
handle color mode changes
c766ffd
@cla-bot cla-bot
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 temporarily deployed to review/4946/merge February 15, 2025 02:50 — with GitHub Actions Inactive
@kimsauce kimsauce changed the title Berry widget demo DOCS-543 - Berry widget demo Feb 17, 2025
@kimsauce kimsauce temporarily deployed to review/4946/merge February 18, 2025 18:19 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge February 20, 2025 04:36 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge February 20, 2025 23:35 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge February 20, 2025 23:36 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge February 22, 2025 00:26 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge March 3, 2025 01:15 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge March 11, 2025 22:50 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge March 16, 2025 00:39 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge March 27, 2025 23:46 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge March 28, 2025 17:00 — with GitHub Actions Inactive
@kimsauce kimsauce temporarily deployed to review/4946/merge April 3, 2025 01:36 — with GitHub Actions Inactive
github-advanced-security[bot]
github-advanced-security bot found potential problems May 7, 2025
View reviewed changes
.github/workflows/review.yml
Comment on lines +7 to +16
uses: SumoLogic/sumologic-documentation/.github/workflows/build_and_deploy.yml@main
with:
hostname: https://d2t1s0ah22jxsa.cloudfront.net
base_url: /${{ github.ref_name }}/
environment: review/${{ github.ref_name }}
secrets:
S3_BUCKET_NAME: ${{ secrets.REVIEW_S3_BUCKET_NAME }}
CLOUDFRONT_DISTRIBUTION_ID: ${{ secrets.REVIEW_CLOUDFRONT_DISTRIBUTION_ID }}
AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}
AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }}

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

Copilot Autofix

AI about 2 months ago

To fix the issue, we will add a permissions block at the root level of the workflow. This block will define the minimal permissions required for the workflow to function. Based on the provided context, the workflow appears to deploy to a review environment, which likely requires read access to repository contents and possibly write access to deployments. We will start with contents: read and add other permissions only if necessary.

Suggested changeset 1
.github/workflows/review.yml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/review.yml b/.github/workflows/review.yml
--- a/.github/workflows/review.yml
+++ b/.github/workflows/review.yml
@@ -4,2 +4,5 @@
 
+permissions:
+  contents: read
+
 jobs:
EOF
@@ -4,2 +4,5 @@

permissions:
contents: read

jobs:
Copilot is powered by AI and may make mistakes. Always verify output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jpipkin1 jpipkin1 Awaiting requested review from jpipkin1 jpipkin1 is a code owner

@JV0812 JV0812 Awaiting requested review from JV0812 JV0812 is a code owner

@mafsumo mafsumo Awaiting requested review from mafsumo mafsumo is a code owner

@amee-sumo amee-sumo Awaiting requested review from amee-sumo amee-sumo is a code owner

@kimsauce kimsauce Awaiting requested review from kimsauce kimsauce is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees

@kimsauce kimsauce

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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chvik @kimsauce