fix: avoid loading Segment analytics when SEGMENT_ID is not set

[seuros]

Added a check to prevent loading the Segment analytics script when SEGMENT_ID is not set.

---

Important

Adds a check in index.html.ejs to load Segment analytics only if SEGMENT_ID is set.

• Behavior:
  • Adds a check in index.html.ejs to load Segment analytics script only if SEGMENT_ID is set.
  • Prevents loading and execution of Segment analytics when SEGMENT_ID is not configured.

^{This description was created by for ad97339. It will automatically update as commits are pushed.}

[welcome]

Welcome to the SigNoz community! Thank you for your first pull request and making this project better. 🤗

[CLAassistant]

All committers have signed the CLA.

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to ad97339 in 23 seconds

More details

• Looked at 154 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. frontend/src/index.html.ejs:104

• Draft comment:
  Consider using consistent indentation for better readability. The current script block has mixed spaces and tabs.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The indentation in the JavaScript code is inconsistent, which can affect readability and maintainability.

2. frontend/src/index.html.ejs:100

• Draft comment:
  Avoid using inline styles in the script tags. Consider moving the script content to an external JavaScript file for better maintainability. This applies to both the Intercom and Segment scripts.
• Reason this comment was not posted:
  Comment was on unchanged code.

Workflow ID: wflow_ygo4D2gqU6gWJpCM

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.