Skip to content

Fix malformed JSX in AdminHome.jsx causing frontend build failure - #23

Merged
udaycodespace merged 1 commit into
udaycodespace:mainfrom
teja-311:fix/adminhome-jsx-structure
Jul 20, 2026
Merged

Fix malformed JSX in AdminHome.jsx causing frontend build failure#23
udaycodespace merged 1 commit into
udaycodespace:mainfrom
teja-311:fix/adminhome-jsx-structure

Conversation

@teja-311

Copy link
Copy Markdown
Contributor

Description

Fixed malformed JSX in frontend/src/components/admin/AdminHome.jsx that broke the frontend build. The dashboard navigation had a duplicated/incomplete <nav> block — the "Activity Logs" button was missing its opening <Button tag and was merged incorrectly with a second <nav> element, splitting what should be a single three-way conditional (Users / Payments / Activity Logs) into two broken conditional blocks.

Merged into a single <nav> with all three buttons properly structured, and combined the conditionals into one clean three-way check (paymentsactivity-logs → default users).

Closes #22

Build Verification

Ran npm run dev in the frontend folder after the fix — Vite starts cleanly with no errors, confirming the build issue is resolved.

Screenshot 2026-07-19 233929

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)

Checklist

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings or console errors.
  • I have tested my changes locally and verified they work as expected.
  • Any dependent changes have been merged and published in downstream modules.

Signed-off-by: Tejashwini <deviteja76@gmail.com>
@udaycodespace
udaycodespace self-requested a review July 20, 2026 03:25
@udaycodespace udaycodespace added ECSoC26-L1 Easy difficulty, auto-assigned by Sentinel — 5 points ECSoC26 Required label for a PR to be eligible for Sentinel scoring good-issue PA-awarded bonus for a well-written, well-scoped issue — +10 XP in review PR is up and waiting on maintainer review and removed frontend labels Jul 20, 2026
@udaycodespace

Copy link
Copy Markdown
Owner

Description

Fixed malformed JSX in frontend/src/components/admin/AdminHome.jsx that broke the frontend build. The dashboard navigation had a duplicated/incomplete <nav> block — the "Activity Logs" button was missing its opening <Button tag and was merged incorrectly with a second <nav> element, splitting what should be a single three-way conditional (Users / Payments / Activity Logs) into two broken conditional blocks.

Merged into a single <nav> with all three buttons properly structured, and combined the conditionals into one clean three-way check (paymentsactivity-logs → default users).

Closes #22

Build Verification

Ran npm run dev in the frontend folder after the fix — Vite starts cleanly with no errors, confirming the build issue is resolved.

Screenshot 2026-07-19 233929 ## Type of Change * [x] 🐛 Bug fix (non-breaking change which fixes an issue)

Checklist

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings or console errors.
  • I have tested my changes locally and verified they work as expected.
  • Any dependent changes have been merged and published in downstream modules.

Build issue addressed as expected. LGTM, approving.

@udaycodespace udaycodespace added bug Something broken or not working as described and removed in review PR is up and waiting on maintainer review labels Jul 20, 2026
@udaycodespace
udaycodespace merged commit 424e3be into udaycodespace:main Jul 20, 2026
4 checks passed
@ecsoc-sentinel ecsoc-sentinel Bot added ECSoC26-L1 Easy difficulty, auto-assigned by Sentinel — 5 points and removed ECSoC26-L1 Easy difficulty, auto-assigned by Sentinel — 5 points labels Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something broken or not working as described ECSoC26-L1 Easy difficulty, auto-assigned by Sentinel — 5 points ECSoC26 Required label for a PR to be eligible for Sentinel scoring good-issue PA-awarded bonus for a well-written, well-scoped issue — +10 XP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Malformed JSX in AdminHome.jsx breaks frontend build

2 participants