fix: 스탬프 완료 로직 버그 수정 (#79)

[hisonghy]

📌 작업 내용 및 특이사항

✅ 스탬프 완료 로직 버그 해결

• 스탬프 완료 요청 시, 200 OK 응답은 잘 되지만 Stamp 엔티티의 completed 필드가 false -> true로 업데이트되지 않는 버그가 발생했습니다.
• 기존 StampService.completeStamp()에 트랜잭션이 적용되어 있는데, 트랜잭션 밖에서 조회된 Stamp 엔티티를 매개변수로 전달받아 업데이트를 수행하고 있었습니다.
• 실행 시 전달된 Stamp는 현재 영속성 컨텍스트에서 관리되지 않는 상태라 더티 체킹이 정상적으로 동작하지 않았습니다.
• StampService.completeStamp()의 트랜잭션을 제거하고, 상위 로직인 StampFacade.completeStamp()에 트랜잭션을 적용해 하나의(동일한) 트랜잭션 내에서 동일한 영속성 컨텍스트로 엔티티가 관리되도록 수정했습니다.
• StampService.completeStamp() 트랜잭션을 유지하고, 매개변수로 ID를 받아 해당 메서드에서 직접 엔티티를 조회하고 업데이트하는 방법도 있었지만 코드의 수정과 변경 범위를 생각해 위의 방법을 사용했습니다.

🌱 관련 이슈

• close 🐛️[BUG]: 스탬프 완료 요청 시 completed 필드가 수정되지 않는 버그 해결 #79

🔍 참고사항(선택)

📚 기타(선택)

[chaiminwoo0223]

고생하셨습니다. 머지 부탁드립니다!