Skip to content

Add Entelligence Script back again #791

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

Closed
wants to merge 5 commits into from

Conversation

fernando-plank
Copy link
Contributor

📥 Pull Request

📘 Description

Entelligence Chat Widget Initialization Improvements

Changes

  • Moved script loading to document head instead of body for better performance
  • Added defer attribute to main script to ensure proper loading order
  • Implemented shadow DOM styles injection with fallback to document head
  • Added interval-based retry mechanism for shadow DOM initialization
  • Improved script loading sequence with proper event handling

Technical Details

  • The widget now loads its styles both in shadow DOM and as fallback in document head
  • CSS is fetched from S3 and injected dynamically
  • Widget initialization waits for DOM content to be loaded
  • Prevents duplicate script injection by checking for existing script tag

🧪 Testing

Please verify:

  • Widget loads correctly on page load
  • Styles are properly applied in both shadow DOM and fallback scenarios
  • The style is breaking the page

@teocns teocns requested a review from areibman March 24, 2025 18:55
Copy link
Contributor

github-actions bot commented Apr 8, 2025

This pull request has been automatically marked as stale because it has not had any activity in the last 14 days.

If no updates are made within 7 days, this PR will be automatically closed.

@github-actions github-actions bot added the stale label Apr 8, 2025
@the-praxs the-praxs closed this Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants