Skip to content

Fix: Added start script in package.json and fix localhost port in README(FIXES GAZ-27)#68

Merged
abhinav-1305 merged 1 commit into
openMF:devfrom
kanishksingh23:dev
Apr 29, 2026
Merged

Fix: Added start script in package.json and fix localhost port in README(FIXES GAZ-27)#68
abhinav-1305 merged 1 commit into
openMF:devfrom
kanishksingh23:dev

Conversation

@kanishksingh23
Copy link
Copy Markdown
Contributor

Problem
The README documented npm start and yarn start to run the application,
but the package.json had no start script defined, causing new contributors
to get an error: Missing script: "start" when trying to run the app.
Additionally, the README referenced http://localhost:3000, but Vite uses
port 5173 by default.
Screenshot 2026-04-19 at 5 56 06 PM

Solution
Added "start": "vite" to the scripts in package.json so npm start works
as documented. Updated the localhost port to 5173 to match Vite's default.

Changes
Added "start": "vite" script to package.json
Updated localhost port from 3000 to 5173 in README.md

Testing
Ran npm start successfully. Dev server started at http://localhost:5173.
Screenshot 2026-04-19 at 6 12 51 PM

@kanishksingh23
Copy link
Copy Markdown
Contributor Author

Hello @DavidH-1 and @abhinav-1305, please review this.

@kanishksingh23
Copy link
Copy Markdown
Contributor Author

@DavidH-1 gentle ping

Copy link
Copy Markdown
Member

@abhinav-1305 abhinav-1305 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@abhinav-1305 abhinav-1305 merged commit 23ba60d into openMF:dev Apr 29, 2026
4 checks passed
@kanishksingh23
Copy link
Copy Markdown
Contributor Author

Thank you, @abhinav-1305, for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants