Skip to content

Require Freshdesk widget URL configuration#1115

Open
rodrcastro wants to merge 5 commits intomainfrom
rodrigo/freshworks-regions
Open

Require Freshdesk widget URL configuration#1115
rodrcastro wants to merge 5 commits intomainfrom
rodrigo/freshworks-regions

Conversation

@rodrcastro
Copy link

@rodrcastro rodrcastro commented Feb 10, 2026

This PR was done with help from Codex.

Summary:

  • update the Freshdesk integration configuration to ask for the full widget URL and explain how to obtain it
  • inject that URL (and its extracted widget ID) directly into the published script so regional domains work correctly
  • refresh manifest metadata, changelog, and build diagnostics to align with the new workflow

Freshdesk now has regions in their widget URLs (ie: https://euc-widget.freshworks.com/widgets/20600000051
8.js) so our previous approach of using just widget.freshwork.com/widgets/{id} wasn't working for some customers. This should allow the user to paste the full URL of the widget from the widget src and load the correct one.

Codex also implemented a backwards-compatible piece to allow widget IDs to still be used and not break it for those that had it working before.

@changeset-bot
Copy link

changeset-bot bot commented Feb 10, 2026

⚠️ No Changeset found

Latest commit: 3dae0f3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 10, 2026

Open in StackBlitz

bun add https://pkg.pr.new/GitbookIO/integrations/@gitbook/api@1115

commit: 3dae0f3

@addisonschultz
Copy link
Collaborator

@rodrcastro Codex made some changes to the CLI I don’t think we want 😓

@rodrcastro
Copy link
Author

@addisonschultz Sorry, didn't notice! Removed that

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