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

[Backend API] Implement endpoint for create event details #213

Open
justinyoo opened this issue Aug 17, 2024 · 2 comments · May be fixed by #281
Open

[Backend API] Implement endpoint for create event details #213

justinyoo opened this issue Aug 17, 2024 · 2 comments · May be fixed by #281

Comments

@justinyoo
Copy link
Contributor

ASSUMPTION: You've logged in as admin

@tae0y
Copy link
Contributor

tae0y commented Aug 28, 2024

#212 OpenAPI 작업완료되어
백엔드 로직구현 찜!

@tae0y
Copy link
Contributor

tae0y commented Aug 31, 2024

진행상황을 공유하기 위해 PR을 제출했지만,
테이블 스토리지 사용도, nosql 사용도 처음이어서 여쭤볼 것이 아주 많습니다!
우선 아래 세 가지입니다.

1. 스토리지 어카운트 bicep 추가방법

스토리지 어카운트를 bicep에 추가했지만 azd up으로 배포되지 않았어요. 우선은 아래 명령어로 생성했습니다.
검색과 코파일럿으로 헤딩해보았지만 해결방법을 찾지 못했습니다.

az storage account create -n tae0ymanulstorage -g rg-tae0y -l koreacentral --subscription ossca

2. 스토리지 커넥션 스트링을 appsettings에서 관리

제출한 PR과 같이 우선 커넥션 스트링을 appsettings에서 참조하는 방식으로 만들었습니다.
앞으로 AppHost에서 ApiApp으로 전달, 애저 배포 파이프라인에서 AppHost로 전달 등
남은 과제는 더 많을 것 같아요.

3. 파티션키와 로우키에 대한 고민

  • Organizer별로 이벤트를 관리할 것이라 생각했고, 파티션키는 OrganizerName, Email을 조합했습니다.
  • 로우키는 EventId로 삼았는데, 어쩌면 제목이나 생성일시가 더 좋은 선택일지 고민이 됩니다.

sikutisa added a commit to sikutisa/azure-openai-sdk-proxy that referenced this issue Sep 3, 2024
sikutisa added a commit to sikutisa/azure-openai-sdk-proxy that referenced this issue Sep 3, 2024
sikutisa added a commit to sikutisa/azure-openai-sdk-proxy that referenced this issue Sep 3, 2024
sikutisa added a commit to sikutisa/azure-openai-sdk-proxy that referenced this issue Sep 3, 2024
sikutisa added a commit to sikutisa/azure-openai-sdk-proxy that referenced this issue Sep 3, 2024
it doesn't use when create TableStorageClient

Related to: aliencube#208, aliencube#213
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants