Skip to content
This repository was archived by the owner on Jun 19, 2024. It is now read-only.

modernizations #19

Open
vasilevich wants to merge 2 commits into
base: master
Choose a base branch
from
Open

modernizations #19

vasilevich wants to merge 2 commits into from

Conversation

@vasilevich
Copy link

@vasilevich vasilevich commented Nov 5, 2019

  1. modified code into a bit more modern js es6 , replacing var with let and const, also some variables were global unnecresarily.
  2. made port 8080 optional, just specify on the command line number eg node api.js 8080 .
  3. added additional api option to convert currency directly like so: api/latest?from=usd&to=eur&amount=5 responds the conversion immediatly

@vasilevich
Update api.js
0e506e8
@vasilevich
modified code into a bit more modern js es6 , replacing var with let …
402045d 
…and const, also some variables were global unnecresarily. made port 8080 optional, just specify on the command line number eg node api.js 8080 . added additional api option to convert currency directly like so: api/latest?from=usd&to=eur&amount=5 responds the conversion immediatly
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vasilevich