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

fix: Change 'api/app.js' redirection to 'localhost:3000/api' #81

Conversation

yunchae-kim
Copy link
Contributor

[#76] Refer to GitHub issue...

This update addresses the following:

Additionally, this update includes the following changes:

  • Reformatted constant imports to use 'import' to align with the ES6 style guide and reordered them based on the import order standard.
  • Replaced some double quotations with single quotations to maintain consistent coding style.
  • Replaced certain 'req' variables used in functions with '_' to resolve ESLint issues related to unused variables.
  • Updated some functions to use ES6 arrow function syntax for improved code readability.

These changes enhance code clarity and maintain consistency with coding standards.

[#76] Refer to GitHub issue...

This update addresses the following:

- Corrected the redirection URL from 'http://localhost:3000' to
'http://localhost:3000/api' to ensure that address redirection functions
correctly. With this change,
'http://ec2-<ip>.compute-1.amazonaws.com/api' will correctly redirect to
the API page as intended.

Additionally, this update includes the following changes:

- Reformatted constant imports to use 'import' to align with the ES6
style guide and reordered them based on the import order standard.
- Replaced some double quotations with single quotations to maintain
consistent coding style.
- Replaced certain 'req' variables used in functions with '_' to resolve
ESLint issues related to unused variables.
- Updated some functions to use ES6 arrow function syntax for improved
code readability.

These changes enhance code clarity and maintain consistency with coding
standards.
[#76] Refer to GitHub issue...

This update changes the HOST variable to redirect api to
localhost:3000/api instead of localhost:3000.
The change was intended to be done in PR-#81 but omitted by mistake,
which is fixed in this PR.
@yunchae-kim yunchae-kim merged commit bd1210f into master Sep 28, 2023
6 checks passed
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.

1 participant