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(recruitment): hide d-day before recruitment #26

Merged
merged 1 commit into from
Apr 25, 2023
Merged

fix(recruitment): hide d-day before recruitment #26

merged 1 commit into from
Apr 25, 2023

Conversation

emost22
Copy link
Member

@emost22 emost22 commented Apr 24, 2023

Summary

d-day가 모집 기간 이전에 노출되는 문제 해결

Issue

변경 전 변경 전 recruitment
변경 후 변경 후 recruitment

@emost22 emost22 added the bug Something isn't working label Apr 24, 2023
@emost22 emost22 requested a review from CODe5753 April 24, 2023 14:29
@emost22 emost22 self-assigned this Apr 24, 2023
@CODe5753
Copy link
Member

CODe5753 commented Apr 24, 2023

image

https://teamnexters.com/recruitment 현재는 디데이가 노출되고 있지 않은데,
캡처본과 어떻게 다른 상황인지 설명 부탁드립니다!

@emost22
Copy link
Member Author

emost22 commented Apr 24, 2023

@CODe5753
현재 서버에서는 #16 에 반영된 코드를 사용 중입니다.
#18 에서 revert로 되돌린 시점은 모집이 공개 시간인 2023.04.24 00:00 되기까지 3시간 28분밖에 남지 않은 상황이어서
다시 코드를 수정하고 pr 올려서 머지되는 과정은 3시간 28분 안에 완료되지 못한다고 판단하여 서버에는 revert 적용을 하지 않았습니다.

@CODe5753
Copy link
Member

현재 서버에서는 #16 에 반영된 코드를 사용 중입니다.
#18 에서 revert로 되돌린 시점은 모집이 공개 시간인 2023.04.24 00:00 되기까지 3시간 28분밖에 남지 않은 상황이어서
다시 코드를 수정하고 pr 올려서 머지되는 과정은 3시간 28분 안에 완료되지 못한다고 판단하여 서버에는 revert 적용을 하지 않았습니다.

네 이 부분도 그럼 롤백 예정인가요?
그럼 기존에 쭉 버전관리 되어오던 이미지 URL과 신규 이미지 URL이 동시에 Drive에서 관리되고 있다고 이해하면 될까요?

@emost22
Copy link
Member Author

emost22 commented Apr 24, 2023

네 이 부분도 그럼 롤백 예정인가요? 그럼 기존에 쭉 버전관리 되어오던 이미지 URL과 신규 이미지 URL이 동시에 Drive에서 관리되고 있다고 이해하면 될까요?

네 그렇게 이해해주시면 됩니다.
서버측 코드도 revert 되면 새로 등록한 구글 Drive의 이미지는 삭제할 예정입니다.

@emost22 emost22 requested a review from CODe5753 April 24, 2023 15:14
Copy link
Member

@CODe5753 CODe5753 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.
말씀해주신 todo 항목들은 이슈에 코멘트 부탁드립니다.

@emost22 emost22 merged commit d0edb34 into main Apr 25, 2023
@emost22 emost22 deleted the fix/d-day branch April 25, 2023 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

모집 시작 이전에 마감일이 노출되는 이슈
2 participants