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

πŸ“¦chore: ν”„λ‘œλ•μ…˜ μ„œλ²„ μœ„μΉ˜ λ³€κ²½ 및 정적 파일 μΆ”κ°€ / μŠ€μ›¨κ±° endpoint λ³€κ²½ #275

Merged
merged 4 commits into from
Dec 3, 2024
Merged

πŸ“¦chore: ν”„λ‘œλ•μ…˜ μ„œλ²„ μœ„μΉ˜ λ³€κ²½ 및 정적 파일 μΆ”κ°€ / μŠ€μ›¨κ±° endpoint λ³€κ²½ #275

merged 4 commits into from
Dec 3, 2024

Conversation

asn6878
Copy link
Member

@asn6878 asn6878 commented Dec 3, 2024

πŸ“‹ μž‘μ—… λ‚΄μš©

ν”„λ‘œλ•μ…˜ μ„œλ²„ μœ„μΉ˜ 변경에 λ”°λ₯Έ 배포 슀크립트 λ³€κ²½

  • ν”„λ‘œλ•μ…˜ μ„œλ²„μ— μ €μž₯μ†Œ 디렉토리가 /root μ—μ„œ /var둜 λ³€κ²½λ˜μ—ˆμŠ΅λ‹ˆλ‹€.

ν”„λ‘œμ νŠΈ λ‚΄ 정적 파일 μ €μž₯μ†Œ μž„μ‹œ μΆ”κ°€

  • ν”„λ‘œμ νŠΈ λ£¨νŠΈλ””λ ‰ν† λ¦¬μ˜ static 디렉토리에 정적 파일이 μΆ”κ°€λ˜μ—ˆμŠ΅λ‹ˆλ‹€.

Tip

이 νŒŒμΌμ— μ ‘κ·Όν•˜κΈ° μœ„ν•΄μ„œλŠ” api.denamu.site/files/{static 디렉토리 λ‚΄ 파일 λͺ…} 으둜 μ ‘κ·Όν•˜μ‹œλ©΄ λ°›μ•„μ˜¬ 수 μžˆμŠ΅λ‹ˆλ‹€!

Warning

μ΄λŠ” μž„μ‹œμ‘°μΉ˜λ‘œ, μΆ”ν›„ S3λ₯Ό ν™œμš©ν•œ 정적저μž₯μ†Œ 뢄리가 진행될 μ˜ˆμ •μž…λ‹ˆλ‹€.
일단은 μž„μ‹œμ μœΌλ‘œ μž‘μ€ νŒŒμΌλ“€λ§Œ 이 경둜λ₯Ό ν†΅ν•΄μ„œ μš”μ²­ν•΄μ£Όμ‹œλ©΄ λ©λ‹ˆλ‹€.

μŠ€μ›¨κ±° λ¬Έμ„œ μ—…λ°μ΄νŠΈ 및 endpoint μˆ˜μ •

  • Swagger에 κ°„λ‹¨ν•œ API νƒœκ·Έλ“€μ΄ μΆ”κ°€λ˜μ—ˆμŠ΅λ‹ˆλ‹€.

Important

κΈ°μ‘΄ endpointκ°€ /api-docs μ—μ„œ /api/swagger 둜 λ³€κ²½λ˜μ—ˆμŠ΅λ‹ˆλ‹€!

@asn6878 asn6878 added the πŸ”§ Chore μ„ΈνŒ… κ΄€λ ¨ label Dec 3, 2024
@asn6878 asn6878 self-assigned this Dec 3, 2024
@netlify Netlify
Copy link

netlify bot commented Dec 3, 2024

βœ… Deploy Preview for denamu canceled.

Name Link
πŸ”¨ Latest commit 1b8b08b
πŸ” Latest deploy log https://app.netlify.com/sites/denamu/deploys/674ee1e652a7c30009b2e0b6

Jo-Minseok
Jo-Minseok approved these changes Dec 3, 2024
View reviewed changes
Copy link
Collaborator

@Jo-Minseok Jo-Minseok left a comment

Choose a reason for hiding this comment

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

LGTM! Cloud λ‚΄ NGINX μ„€μ • νŒŒμΌλ„ static 디렉토리 경둜 λ°”κΎΈμ…¨λ‚˜μš”?

@asn6878
Copy link
Member Author

asn6878 commented Dec 3, 2024

LGTM! Cloud λ‚΄ NGINX μ„€μ • νŒŒμΌλ„ static 디렉토리 경둜 λ°”κΎΈμ…¨λ‚˜μš”?

λ„΅! https://api.denamu.site/files/images.png μ ‘κ·Όν•΄λ³΄μ‹œλ©΄ 이미지 잘 μ˜¬κ±°μ—μš” γ…Žγ…Ž

CodeVac513
CodeVac513 approved these changes Dec 3, 2024
View reviewed changes
Copy link
Collaborator

@CodeVac513 CodeVac513 left a comment

Choose a reason for hiding this comment

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

LGTM!

@asn6878 asn6878 merged commit 89fa7d2 into main Dec 3, 2024
1 check passed
@asn6878 asn6878 deleted the chore/static-files-swagger-endpoint branch December 5, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jo-Minseok Jo-Minseok Jo-Minseok approved these changes

@CodeVac513 CodeVac513 CodeVac513 approved these changes

@junyeokk junyeokk junyeokk approved these changes

@jungmyunggi jungmyunggi Awaiting requested review from jungmyunggi

Assignees

@asn6878 asn6878

Labels
πŸ”§ Chore μ„ΈνŒ… κ΄€λ ¨
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@asn6878 @junyeokk @CodeVac513 @Jo-Minseok