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

[Refactor] change temp user with parsing current user #41

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

Eeap
Copy link
Contributor

@Eeap Eeap commented Dec 3, 2023

Description

I have changed logic to find user with parsing current user.

Also, there is something error about parsing body. You should delete depends like this free_in: frees.FreeCreate=Depends(). And if you have files data in query, you have change with File() like this files: List[UploadFile] = File(...),

Related Issue

Closes #40

@Eeap Eeap requested review from nahyun0121 and dusdjhyeon December 3, 2023 05:47
@Eeap Eeap self-assigned this Dec 3, 2023
@Eeap Eeap added the refactor label Dec 3, 2023
@Eeap Eeap changed the title refactor: change temp user with parsing current user [Refactor] change temp user with parsing current user Dec 3, 2023
Copy link
Contributor

@dusdjhyeon dusdjhyeon left a comment

Choose a reason for hiding this comment

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

LGTM👍

@Eeap Eeap merged commit 67c4403 into dev Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] change temp user data with parsing current user
2 participants