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

review done #39

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

review done #39

wants to merge 1 commit into from

Conversation

wlwkgus
Copy link

@wlwkgus wlwkgus commented Feb 14, 2022

✔ Issue 번호

Closes #


✔ 해결한 내용

  • (가능한 자세히 작성해주기)

✔ 추가된 코드의 기능/파일명 목록

  • (추가된 기능/파일명을 적기. 없다면 문항 지우기!)

✔ 필요한 후속작업

  • (없다면 문항 지우기!)

✔ 참고한 자료

  • (없다면 문항 지우기!)

✅ 셀프 체크리스트

  • Reviewers / Assignees / Labels / Projects 지정했나요?
  • Merge 하는 브랜치가 올바른가요?
  • 커밋 메세지를 컨벤션에 맞게 작성했나요?
  • 변경 후 코드를 컴파일/런서버 했을 때 에러가 발생하지 않나요?
  • PR 내용과 관련없는 변경사항은 없나요?
  • 자신의 코드에 대한 검토가 이뤄졌나요?

@@ -32,28 +32,29 @@ def myInfo(request):


def mainSearch(request):
# TODO : arr 변수 이름 명확하게 바꾸기
arr = ['','','','']
len = HashTag.objects.count()
Copy link
Author

Choose a reason for hiding this comment

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

len : 기본적인 파이썬 내장 함수

len()을 사용 못할 수 있음

continue
else:
arr[count]=random_object.name
count += 1;
print(arr)
print(arr)
Copy link
Author

Choose a reason for hiding this comment

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

return HashTag.objects.orderbyRandom[:4]

req = json.loads(request.body)
title = req['music']
artist = req['artist']

return JsonResponse({'music': title, 'artist': artist})

Copy link
Author

Choose a reason for hiding this comment

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

@login_required

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant