Skip to content

feat(zendesk): load messenger from app entry#257

Merged
petar-omni merged 2 commits into
mainfrom
feat/zendesk
May 11, 2026
Merged

feat(zendesk): load messenger from app entry#257
petar-omni merged 2 commits into
mainfrom
feat/zendesk

Conversation

@petar-omni
Copy link
Copy Markdown
Collaborator

Initialize Zendesk once from the application entrypoint and keep widget address updates flowing through the shared singleton.

@petar-omni petar-omni requested a review from Philippoes May 4, 2026 15:00
@aws-amplify-eu-central-1
Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-257.d24ijdjz9ynxxy.amplifyapp.com

@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 8, 2026

@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 8, 2026

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
Obfuscated code: npm @stakekit/widget is 90.0% likely obfuscated

Confidence: 0.90

Location: Package overview

From: package.jsonnpm/@stakekit/widget@0.0.270

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@stakekit/widget@0.0.270. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

dnehl
dnehl previously approved these changes May 11, 2026
Copy link
Copy Markdown

@dnehl dnehl left a comment

Choose a reason for hiding this comment

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

Overall good - just a small improvment for the magic string, and whats about the key?

Comment thread src/app/widget/widget.tsx Outdated
Comment thread src/app/zendesk.ts
Copy link
Copy Markdown

@dnehl dnehl left a comment

Choose a reason for hiding this comment

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

nice!

@petar-omni petar-omni merged commit f161c09 into main May 11, 2026
6 checks passed
@petar-omni petar-omni deleted the feat/zendesk branch May 11, 2026 10:54
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.

2 participants