Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/issue 9 backend development #17

Closed

Conversation

mong00x
Copy link
Collaborator

@mong00x mong00x commented Aug 11, 2024

Description:

This PR sets up the foundational backend structure for the project, addressing the initial requirements outlined in issue #9. The following tasks have been completed:

Changes Made:

Installed Dependencies:

  • Installed express as the primary framework for building the API server.
  • Installed nodemon (with the --force flag) to facilitate automatic server restarts during development. The --force flag was necessary due to outdated package dependencies.
  • Note: Despite using the --force flag during the installation of nodemon, no other parts of the project were affected. The installation was successful, and all existing functionality remains intact.

Updated package-lock.json:

Created server.js:

  • Added a server.js file within a newly created server directory.
  • The server.js file contains a basic Express server setup that listens on port 3001.
  • The server currently includes a simple root route (/) that returns "Hello World!" as a response.

Added server Script to package.json:

  • Added a new npm script called "server" in the package.json.
  • This script runs nodemon server/server.js, enabling the server to automatically restart when changes are detected during development.

Testing:

  • The server has been manually tested to ensure it runs correctly on http://localhost:3001 and that the nodemon script works as expected.

Next Steps:

  • Once this PR is reviewed and merged, the next step will be to start adding specific API endpoints and handling additional configurations as needed.

- Replaced `Input` with `InputNumber` from `antd` for the weight input field to utilize built-in validation for non-negative numbers.
- Removed the input validation logic from useEffect for better code structure and readability.
 - Specified min and max prop for InputNumber for validation
Copy link

vercel bot commented Aug 11, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fitness-playground ❌ Failed (Inspect) Aug 11, 2024 10:33am

@mong00x
Copy link
Collaborator Author

mong00x commented Aug 11, 2024

Just realised this PR has 2 unwanted commits b9b58e0 and 3487937. To ensure the PR is clear and focused on its addressing issue, I’m requesting that this PR be closed

I’ll be opening a new PR with the correct commits shortly after. This new PR will still address the initial setup for the backend, but without the unnecessary commits.

Thank you.

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