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] 파일 업로드 에러 / 쪽지 전송 에러 및 UX 개선 / QA 메일 템플릿 적용 #604

Merged
merged 5 commits into from
Jun 26, 2024

Conversation

Junanjunan
Copy link
Contributor

@Junanjunan Junanjunan commented Jun 18, 2024

변경 사항

  • REST API를 통한 파일 업로드 요청의 경우 None 값이 포함되는 경우 None 에서 filename을 찾는 오류 발생
    -> 파일이 존재하는 경우만 검증로직을 수행하도록 수정
  • 쪽지 전송 포인트 중복 차감 에러 수정, 문구 변경
  • 특정 회원 지정 쪽지 전송시, input 창에 유저 정보가 입력되도록 UX개선
  • QA 답변 알람 메일 템플릿 적용

관련 이슈

#609
#610

기타 정보

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요. PR을 보내기 전에 모든 항목을 확인해야 합니다.

  • 동일한 업데이트/변경에 대한 다른 Pull Requests가 열려있는지 확인했습니다.
  • 테스트가 성공적으로 수행되었는지 확인했습니다.

REST API를 통한 파일 업로드 요청의 경우 None 값이 포함될 수 있으므로 예외처리
@KimTom89 KimTom89 added the bug Something isn't working label Jun 19, 2024
@Junanjunan Junanjunan changed the title [fix] 파일 업로드 > file 존재시에만 검증 로직 진행 [fix] 파일 업로드 에러 / 쪽지 전송 에러 및 UI 개선 / QA 메일 템플릿 적용 Jun 24, 2024
@Junanjunan Junanjunan changed the title [fix] 파일 업로드 에러 / 쪽지 전송 에러 및 UI 개선 / QA 메일 템플릿 적용 [fix] 파일 업로드 에러 / 쪽지 전송 에러 및 UX 개선 / QA 메일 템플릿 적용 Jun 24, 2024
Copy link
Collaborator

@KimTom89 KimTom89 left a comment

Choose a reason for hiding this comment

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

@Junanjunan

추가 오류 발생

받은쪽지 상세페이지 > 답장 기능에서 회원 아이디가 정상적으로 출력되지 않음

수정 코드

1. memo_form.html

{{ request.query_params.me_recv_mb_id }} -> {{ target.mb_id }} (원복)

2. bbs/memo.py

@router.get("/memo_form", dependencies=[Depends(get_login_member)])
async def memo_form(
    request: Request,
    member_service: Annotated[MemberService, Depends()],
    memo_service: Annotated[MemoService, Depends()],
    me_id: int = Query(default=None),
    me_recv_mb_id: str = Query(default=None),
):
    """
    쪽지 작성 페이지
    """
    # 답장할 쪽지 & 회원 정보 조회
    target = None
    memo = memo_service.fetch_memo(me_id)
    if me_id and memo:
        target = member_service.read_member(memo.me_send_mb_id)
    elif me_recv_mb_id:
        target = member_service.fetch_member_by_id(me_recv_mb_id)

    context = {
        "request": request,
        "target": target,
        "memo": memo,
    }
    return templates.TemplateResponse("/memo/memo_form.html", context)

@KimTom89 KimTom89 merged commit b448370 into gnuboard:master Jun 26, 2024
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