brave-people / Dev-Event-Client

🎉🎈 Dev Event Client - 관리자 🌐
https://dev-event-client.vercel.app/
MIT License
3 stars 3 forks source link

베너 관리자 페이지 기획 검토 #151

Closed KoEonYack closed 1 year ago

KoEonYack commented 1 year ago

@leedoopal 반가워~~~

1. 배경

모바일에서만 제공하는 배너를 웹과 동시에 제공하려고 하는데, 그러기 위해서는 관리자 페이지의 문구 수정과 배너 배경 색상 선택 관련 기능 추가가 필요해서 신규 기획을 만들게 되었어.

2. 활용처

image

신규 개편 작업중인 WEB, 빨간 영역에 들어가게 될 거야.

image

모바일 배너 영역

안드로이드 앱의 경우 막바지 작업이 남았고, 곧 출시를 앞두고 있어. 배너 기능을 잘 만들어주어서 안드로이드 앱에는 배너 기능이 들어가서 출시 될 수 있게 되었어. 고마워!!!

3. 기획서


잘 지내고 있어~?

얼마전에 침착맨이 우리 회사와서 촬영하고 갔는데 유툽에 올라왔어! 나는 카메라 근처에 얼쩡거렸는데 편집되었구!

https://youtu.be/vk-xYUp0WtI 많관부!

leedoopal commented 1 year ago

@KoEonYack 헉 미안..!! 요즘 갑자기 바빠져서 확인을 늦게했네ㅠ.ㅠ 기획해준거 좋은거같아! 그렇게 만들어볼게 고마워~!!!!


ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 어딨지?? 하면서 유심히 보다가 방긋 모습이 바로 보였어ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 소식 고마워ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

KoEonYack commented 1 year ago

@leedoopal

엄청난 ㅋ이다!!

확인 고마워! 주말에 API 잘업해볼께~!!

leedoopal commented 1 year ago

@KoEonYack 프론트 작업은 다 끝났어!! API 작업 완료되면 말해줘!~!!!

KoEonYack commented 1 year ago

@leedoopal 엇,,,!! 로켓 작업해주었는데 내가 알림을 놓쳤네 ㅠㅠ,,, API 연동해서 로컬 테스트 마치고 알려줄께!!

KoEonYack commented 1 year ago

@leedoopal

로컬에서 어드민 올려봤는데 어떻게 이렇게 찰떡같이 원하는대로 만들어줄 수 있는거야~?! 정말 대단해~~~~!!! 👍

API는 개발 환경에 배포했어~!! 테이블을 안전하게 신규 테이블로 만들어서 기존에 개발에 등록한 배너는 안보일꺼야~ 테스트를 하려면 다시 등록해야햇!

  1. 어드민 URI에 mobile이 제거했어~
  2. backgroundColor 필드가 생겼고 필수 필드야!

스크린샷 2023-09-02 오후 3 30 11


의견!

image

노출여부는 default checked였으면 좋겠어!! 이것은 예전에 말하고싶었는데!! 이번에 바꾸는김에 하면 좋을듯!!


image

모바일 -> 최상단으로 문구가 변경하면 될것같아!


image

color picker가 너무 하단에 치우쳐져있어!!


image

가끔 색상이 엉뚱한게 보일때가 있는데 재현은 안되네 ㅠㅠ,,,


leedoopal commented 1 year ago

@KoEonYack 헉 고마워!!!! 너무 기분좋아 🍰 ~!! 자세한 피드백도 고마워!! 로컬에서 돌려볼거라고 생각을 못했는데 대단해~~!! 👍

그런데 api 요청 보내봤는데 에러가 발생하네;; 왜 그런지 알 수 있을까??

POST admin/v1/banner/top request body

{"title":"테스트","eventLink":"11","priority":1,"visibleYn":"Y","startDateTime":"2023-09-05T00:00","endDateTime":"2023-09-26T00:00","bannerImage":"https://brave-people-3.s3.ap-northeast-2.amazonaws.com/dev/DEVEVENT/2023-09-05-00-15-3950-a34564dd.png","backgroundColor":"#03878c"}

error

스크린샷 2023-09-05 오전 12 17 31

원래 필드 네임이 _ 형태였는데 요번에 스웨거 보니 전부 카멜로 바뀐 것 같아! 다 바꾸면 될까??

leedoopal commented 1 year ago

요거는 수정했어! 버그는 잘 재현이 안되네;; 혹시 케이스를 찾으면 알려줘! ㅠ.ㅠ

KoEonYack commented 1 year ago

@leedoopal

수정 넘 고마워 >0<


{
    "title": "테스트",
    "event_link": "11",
    "priority": 1,
    "visible_yn": "Y",
    "start_date_time": "2023-09-05T00:00",
    "end_date_time": "2023-09-26T00:00",
    "banner_image": "https://brave-people-3.s3.ap-northeast-2.amazonaws.com/dev/DEVEVENT/2023-09-05-00-15-3950-a34564dd.png",
    "background_color": "#03878c"
}

스크린샷 2023-09-05 오후 12 09 28

이렇게 실행해야해~ 스네이크 케이스로 보내는것은 여전해! 그러고보니 배너 생성 응답값을 200 상태코드로 응답하네 ㅜㅠ,, 201 CREATED로 수정해야게써..


에러 메시지에 카멜 케이스로 메시지가 내려오는것은 스프링부트에서 지정된 기본 메시지여서 그래~ 이것을 커스텀하려면 메시지를 필드마다 지정해야하는데,, 상당한 귀차니즘에 빠져서 ㅠㅠ,,, 우리끼리는 스네이크로 보내야한다고 정하면 되니깐!! 지나고보니 스네이크보단 카멜로 했어야했는데 생각도 들고 ; ㅅ ;

leedoopal commented 1 year ago

@KoEonYack 아하 그렇구나~~!! ㅋㅋㅋㅋ아냐 이유 알려줘서 고마워! dev에다가 올려놨어 😸

KoEonYack commented 1 year ago

@leedoopal

스크린샷 2023-09-07 오전 12 35 55 스크린샷 2023-09-07 오전 12 36 02

아직 모바일이라는 표현이 남아있다!!

서버에서 배너 등록할때 응답값을 201 status code로 변경한다면 프론트에서 코드 변경이 필요할까~~?!

leedoopal commented 1 year ago

@KoEonYack 헉 모바일 > 최상단으로 전부 수정했어!~!

응답값을 변경하게 되면 프론트에서도 코드를 바꿔줘야하는건 맞는데 지금 배너 생성하면 201로 내려오는 것 같은데?! 프론트에서는 예전부터 배너 생성시 status_code 201이라면 -> 성공으로 처리하고 있었어..!

스크린샷 2023-09-07 오후 11 45 05
KoEonYack commented 1 year ago

@leedoopal

너무 늦었지...? 😅

해당 작업건 리얼에 배포 완료했어!

웹 혹은 ios 런칭하면 배너 제작 과제랑 함께 배포하려고 했는데..

늦어져서 개발 서버에만 배포하고 있는것보다 일단 리얼에 배포하는게 좋을것 같아서 리얼에 배포했어..!! (간간히 안드 앱에서 슬슬 하나씩 등록해볼 예정!!)

엄청 빨리 만들어주어서 고마워! 색상 선택하는것은 아무리 생각해봐도 넘 잘 만들어진것 같아!! 넘 고마워!!!

KoEonYack commented 1 year ago

@leedoopal

리얼 배포해주었네~~!! 넘 고마워!!! 👍 💯

leedoopal commented 1 year ago

@KoEonYack 분명 api 리얼 배포했다는 메일을 받았는데.. 내가 리얼에 배포를 안했더라구;;!! 부랴부랴 배포했어ㅠ.ㅠ 고마워 확인해줘서!!