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

[GET] /myroom/:roomId 특정 습관방 사진 모아보기 #66

Merged
merged 7 commits into from
Jan 16, 2022

Conversation

xxeol2
Copy link
Contributor

@xxeol2 xxeol2 commented Jan 16, 2022

✅ Default Checklist

  • check branch

  • set Labels

  • set Reviewers


📕 Task

  • 특정 습관방 사진 모아보기

😳 Issue

  • 사진 리사이징

@xxeol2 xxeol2 added Enhancement ✨ New feature or request GET 🧡 HTTP method : GET 🐱 설희 labels Jan 16, 2022
@xxeol2 xxeol2 self-assigned this Jan 16, 2022
@@ -0,0 +1,93 @@
const functions = require('firebase-functions');
const admin = require('firebase-admin');
Copy link
Member

Choose a reason for hiding this comment

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

@xxeol2
설희쌤 요녀석 안쓰이는거 같아요~~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

삭제했어요!

const statusCode = require('../../../constants/statusCode');
const responseMessage = require('../../../constants/responseMessage');
const db = require('../../../db/db');
const { userDB, roomDB, sparkDB, recordDB } = require('../../../db');
Copy link
Member

Choose a reason for hiding this comment

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

userDB에도 접근 안하는거 같아용~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

삭제했어요!

Comment on lines -28 to +29
GET_WAITROOM_DATA_IMPOSSIBLE: '참여할 수 없는 코드예요.',
GET_WAITROOM_DATA_NULL: '존재하지 않는 코드예요.',
GET_WAITROOM_DATA_IMPOSSIBLE: '참여할 수 없는 코드에요.',
GET_WAITROOM_DATA_NULL: '존재하지 않는 코드에요.',
Copy link
Member

Choose a reason for hiding this comment

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

이거 머지하면서 지우신게 제가 수정한 내용인데, 요거 "예요"가 맞는 문법이라구 하네용!
추가하신 내용 삭제하시구, 지우신 내용으로 대체 해주시면 될거같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정했어요!

@xxeol2 xxeol2 merged commit 1f7725a into develop Jan 16, 2022
@xxeol2 xxeol2 deleted the feature/#56 branch January 16, 2022 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ✨ New feature or request GET 🧡 HTTP method : GET 🐱 설희
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants