fix: conditionally render Intercom script in EJS based on APP_ID

[seuros]

Added EJS conditional logic to render the Intercom script tag only if INTERCOM_APP_ID is defined

---

Important

Conditionally renders Intercom script in index.html.ejs based on INTERCOM_APP_ID.

• Behavior:
  • Conditionally renders Intercom script in index.html.ejs only if INTERCOM_APP_ID is defined.
  • Prevents loading Intercom script when INTERCOM_APP_ID is not set.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 4fdaaf2 in 15 seconds

More details

• Looked at 94 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:62

• Draft comment:
  The indentation of the EJS conditional block is inconsistent. Consider aligning it with the rest of the HTML structure for better readability.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The indentation of the EJS conditional block is inconsistent with the rest of the file.

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

• Draft comment:
  Avoid using inline styles in HTML. Consider using external stylesheets or CSS classes instead. This applies to the entire script block.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR does not violate any of the specified rules. The changes are related to conditional rendering of a script based on a variable, which is appropriate for the context.

Workflow ID: wflow_2EWc9BOX51dN4HQJ

---

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