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/seasearch: add wiki search api #6606

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cir9no
Copy link
Contributor

@cir9no cir9no commented Aug 21, 2024

No description provided.

@cir9no cir9no force-pushed the feat/seasearch-add-wiki-search-sup branch from 7cf89c8 to 16066c4 Compare August 27, 2024 09:47
@cir9no cir9no force-pushed the feat/seasearch-add-wiki-search-sup branch from 16066c4 to cc03cd1 Compare September 25, 2024 07:46
@@ -313,6 +313,24 @@ def get_search_repos(username, org_id):

return repos

def get_search_wiki_ids(username, org_id):
owned_wikis, shared_wikis, group_wikis, public_wikis = get_user_repos(username, org_id=org_id)
Copy link
Contributor

@JoinTyang JoinTyang Sep 30, 2024

Choose a reason for hiding this comment

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

变量名不对,现在查出来的是资料库,还不是wiki, 下面最好也别用wiki_id,因为实际上并不是wiki_id 而是repo_id

error_msg = 'search_wiki invalid.'
return api_error(status.HTTP_400_BAD_REQUEST, error_msg)

if search_wiki == 'all':
Copy link
Contributor

Choose a reason for hiding this comment

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

这部分可以去掉了

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.

2 participants