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

[feat] 어드민 이벤트 생성/수정 기능 구현 (resolve #29) #109

Merged
35 commits merged into from
Aug 19, 2024

Conversation

lybell-art
Copy link
Collaborator

@lybell-art lybell-art commented Aug 17, 2024

#️⃣ 연관 이슈

📝 작업 내용

어드민 이벤트 생성/수정 기능을 구현했습니다.

  • 모든 상태를 리듀서로 관리하고 있습니다. 이유는... 상태가 엄청나게 많아서 그래요. 시간 관련된 것들은 여러 상태가 엮여 있기도 합니다.
  • 예외적으로, 저희 DB설계의 끔찍한 혼종인 이벤트 프레임은 현재로선 텍스트 입력으로 구현했습니다.
  • 예외적으로, 화면 이탈 시 alert 창 뜨는 건 해당 기능을 도와주는 react-router-dom의 data api로 마이그레이션을 해야 하기 때문에 많이 힘듭니다.

폰트를 수정했습니다.

  • 얇은 폰트인 레귤러 폰트를 추가했습니다.
  • 그에 따라, 폰트를 지정 안 해 줘서 레귤러로 나오던 폰트들은 싹 다 지정해 주었습니다.
    • 워낙에 광범위하게 바꿨기 때문에, 컨플릭트가 날 수 있습니다.

약간의 사용성 및 UX 개선이 있었습니다.

  • 버튼에 이미지 아이콘이 존재할 때, 해당 이미지 아이콘을 드래그가 불가능하도록 바꿨습니다. 이제 여러 번 클릭해도 브라우저의 네이티브 드래그 동작을 유발하지 않습니다.
  • 이제 모든 환경에서 주행거리 인터랙션의 정답을 맞힐 수 있습니다. (전에는 1920x1080같이 매우 와이드한 화면이 아니면 주행거리 인터랙션의 정답에 근접하지를 못했음 )
  • 이제 보조금 이벤트의 동전 클릭이 더 쉬워집니다. (전에는 가장자리에서 동전을 클릭하면 연속으로 클릭이 안 됐음)
  • 이제 보조금 이벤트의 동전을 여러 번 클릭하면 로티가 여러 개 생깁니다. 즉, 동전 파티클이 폭포수처럼 쏟아집니다!

약간의 버그 수정이 있었습니다.

  • fetchServer 함수는 이제 빈 응답에 대응합니다.
    • 전에는 서버에서 빈 응답을 받을 때 json으로 파싱을 못 하는 버그가 있었습니다.
    • 백엔드에서 빈 응답을 반환하는 api가 몇 개 있었습니다.

참고 이미지 및 자료

스크린샷 2024-08-18 오전 1 13 24

-[fix] 일괄설정 체크 해제 시에도 일괄설정 시각 폼이 유지되게 변경
[refactor] 삭제버튼공통컴포넌트로분리:
@lybell-art lybell-art added fix 버그 및 오류 수정 feat 기능 구현 labels Aug 17, 2024
@lybell-art lybell-art requested a review from darkdulgi August 17, 2024 16:26
@lybell-art lybell-art self-assigned this Aug 17, 2024
@darkdulgi darkdulgi closed this pull request by merging all changes into dev in 2f0c4bf Aug 19, 2024
@lybell-art lybell-art deleted the feature/29-admin-event-create branch August 28, 2024 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 구현 fix 버그 및 오류 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants