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.

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

init order service #443

Merged
merged 12 commits into from
Mar 20, 2023
Merged

init order service #443

merged 12 commits into from
Mar 20, 2023

Conversation

LEHOANGGLAM
Copy link
Contributor

@LEHOANGGLAM LEHOANGGLAM commented Mar 17, 2023 •
edited
Loading

Please review this PR for me.

  • Init OrderSerivce with port: 8085
  • Create github action
  • Add order service in prometheus for tracking
  • Add port 8085 to realm-export.json
  • Add order location in nginx
  • Add query for create order database in postgres_init
    Please tell me if there any missing, thank you.

chanhnguyen6975
chanhnguyen6975 reviewed Mar 17, 2023
View reviewed changes
order/src/main/resources/application.properties Outdated

# swagger-ui custom path
springdoc.swagger-ui.path=/swagger-ui
springdoc.packagesToScan=com.yas.rating
Copy link
Contributor

Choose a reason for hiding this comment

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

order

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks

@sonarcloud
Copy link

sonarcloud bot commented Mar 20, 2023

[order] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@LEHOANGGLAM LEHOANGGLAM merged commit 0bd07f5 into main Mar 20, 2023
@LEHOANGGLAM LEHOANGGLAM deleted the init-order-service branch March 20, 2023 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@toannguyenk toannguyenk toannguyenk left review comments

@chanhnguyen6975 chanhnguyen6975 chanhnguyen6975 approved these changes

@thiennn thiennn thiennn approved these changes

@khanhtrand khanhtrand Awaiting requested review from khanhtrand

@DinhThaiVV DinhThaiVV Awaiting requested review from DinhThaiVV

@nhientrand nhientrand Awaiting requested review from nhientrand

@DieuNguyenThi DieuNguyenThi Awaiting requested review from DieuNguyenThi

@khoahd7621 khoahd7621 Awaiting requested review from khoahd7621

@hvantran hvantran Awaiting requested review from hvantran

@kiettrangthuan kiettrangthuan Awaiting requested review from kiettrangthuan

@lnghia lnghia Awaiting requested review from lnghia

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@LEHOANGGLAM @thiennn @chanhnguyen6975 @toannguyenk