Lidya-w2-Node.js #16

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.

Sign up for GitHub

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

Jump to bottom

Open

lidyaT21 wants to merge 4 commits into HackYourAssignment:main from lidyaT21:Lidya-w2-Node.JS

lidyaT21 commented Feb 25, 2025

Added OpenWeatherMap API integration
Created sources/keys.js to store API key
Implemented fetch() request inside the POST route
Returns city name & temperature if found, otherwise sends error message
Installed Jest & Supertest as dev dependencies
Added test script in package.json
Created tests/app.test.js with a basic test
Configured Babel with babel-jest & @babel/preset-env
Add API test cases for city validation & responses

lidyaT21 added 4 commits

February 22, 2025 18:46


          finshed

6fd07f3


          updated

1223e4d


          Lidya-w2-Node.Js

d7fe51c


          Lidya-w2-nodeJs

9febb58

saadkhaleeq610 approved these changes

View reviewed changes

assignments/hackyourtemperature/sources/Keys.js

saadkhaleeq610 Mar 8, 2025

exposing api key is a security risk. You should consider using dotenv module and store api keys in .env file in the root directory of the project.

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

Labels

None yet