Updated Leftnav.jsx

[bharathk08]

Summary by CodeRabbit

• Chores
  • Updated image loading to use remote URLs instead of locally bundled assets.

[coderabbitai]

📝 Walkthrough

Walkthrough

Removed a local image import from LeftNav.jsx and replaced it with a hard-coded remote S3-hosted image URL in the component renderer. The update affects only the image source reference with no changes to markup or behavior.

Changes

Cohort / File(s)	Summary
Image Asset Migration src/components/LeftNav.jsx	Removed local book.png import and updated <img> element's src attribute to point to a remote S3-hosted URL.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

• Abdul-jailani
• Santhosh-testsigma

Poem

🐰 A book once lived local, snug on the drive,
Now floats on the cloud, perfectly alive!
One import removed, one URL stays,
Remote assets bright, in S3's haze ☁️📚

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title is vague and generic, using only the filename without describing the specific change made (replacement of local image import with remote S3 URL).	Revise the title to be more specific about the change, e.g., 'Replace local book image import with S3 URL in LeftNav' to clearly communicate the primary modification.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch dev

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

src/components/LeftNav.jsx (1)

7-7: Remove the commented-out import.

Line 7 leaves dead code in place after the change. Please delete it to keep the file clean.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/components/LeftNav.jsx` at line 7, Remove the dead commented-out import
statement "import book from './images/book.png';" from the LeftNav.jsx file so
there is no leftover commented import; simply delete that line to keep the
component clean and ensure no unused/commented imports remain in the LeftNav
component.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/components/LeftNav.jsx`:
- Line 197: The img tag in LeftNav.jsx currently uses a hard-coded S3 URL which
bypasses centralized asset configuration; replace that literal URL with the
configured asset host or bucket value (e.g., read from the site config or
environment variable used in your Gatsby setup such as the GATSBY_ or
process.env asset var) so the src is constructed from the centralized config,
and update the component to import or reference that config (LeftNav.jsx img
src) rather than embedding the S3 host directly.

---

Nitpick comments:
In `@src/components/LeftNav.jsx`:
- Line 7: Remove the dead commented-out import statement "import book from
'./images/book.png';" from the LeftNav.jsx file so there is no leftover
commented import; simply delete that line to keep the component clean and ensure
no unused/commented imports remain in the LeftNav component.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f260aaa6-c540-45d3-b7a1-18018fa82a14

📥 Commits

Reviewing files that changed from the base of the PR and between 3dcf044 and e9f3358.

📒 Files selected for processing (1)

• src/components/LeftNav.jsx

[coderabbitai]

⚠️ Potential issue | 🟠 Major

Avoid hard-coded S3 URLs in component markup.

Line 197 hard-codes a specific bucket host, which bypasses your centralized asset configuration pattern (gatsby-config.js Line 1-6 and Line 82-90, plus .env Line 10-12). This creates environment-coupling and higher breakage risk when host/bucket values change.

Suggested direction

-        <img src='https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/book.png' alt='book' />
+        <img src={book} alt='book' />

-// import book from './images/book.png';
+import book from './images/book.png';

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/components/LeftNav.jsx` at line 197, The img tag in LeftNav.jsx currently
uses a hard-coded S3 URL which bypasses centralized asset configuration; replace
that literal URL with the configured asset host or bucket value (e.g., read from
the site config or environment variable used in your Gatsby setup such as the
GATSBY_ or process.env asset var) so the src is constructed from the centralized
config, and update the component to import or reference that config (LeftNav.jsx
img src) rather than embedding the S3 host directly.