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] 피그마 파일 추가 시 파일 이름까지 저장하도록 변경 #55

Merged
merged 3 commits into from
Nov 16, 2024

Conversation

iiqcov
Copy link
Contributor

@iiqcov iiqcov commented Nov 16, 2024

#️⃣ 관련 이슈

💡 작업내용

피그마 로그인 시 리프레시 토큰까지 저장되도록 수정

  • 이후 필요 시 피그마 api를 이용할 수 있도록 하기 위해 추가하였습니다.

Figma 도메인서비스와 api 서비스 분리

  • figma 레포지토리에 대한 조작을 진행하는 도메인 서비스와 외부 api를 이용하는 api 서비스를 분리하였습니다.

파일 id 헤더 사용 시 파일 이름까지 저장하도록 변경

  • 피그마 파일에 저장된 리프레시 토큰으로 엑세스토큰을 발급받고 그 엑세스토큰을 이용하여 파일 이름을 받을 수 있도록 설정하였습니다.

📸 스크린샷(선택)

📝 기타

  • 이후 피그마 도메인을 아예 분리해도 좋을 것 같습니다

@iiqcov iiqcov requested a review from cowboysj November 16, 2024 07:05
@iiqcov iiqcov self-assigned this Nov 16, 2024
Copy link
Member

@cowboysj cowboysj left a comment

Choose a reason for hiding this comment

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

수고하셨습니당..! 파일 분리하니까 훨씬 깔끔하네요..!

@iiqcov iiqcov merged commit 1c3c4ca into develop Nov 16, 2024
2 checks passed
cowboysj pushed a commit that referenced this pull request Nov 16, 2024
* fix : 로그인 시 피그마 리프레시 토큰까지 저장

* fix : 동일 피그마 계정이 중복 저장되지 않도록 수정

* feat : 파일 id 헤더 사용 시 파일 저장 로직 변경
@cowboysj cowboysj added the ✨ feature 기능 구현 label Nov 16, 2024
@iiqcov iiqcov deleted the feat/#54 branch November 23, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants