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

Add support for adv_search #4

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hugoprudente
Copy link

No description provided.

@hugoprudente
Copy link
Author

I'm not adding the documentation as I would like to confirm if it's generated manually or it's a command. If so can you provide me guidance on it?

@Rocker2234
Copy link
Owner

Hello Hugo, thank you for adding more relevant info. As for the documentation you can simply add the feature description and how to use on the docs/source/doc/InfoPage.md. Please follow a similar way which already exists on that file. Since you have only added 2 new attributes, just add them in the attributes section.

@Rocker2234
Copy link
Owner

And as for adv_search, you can document it in the docs/source/doc/Searching.md. This project uses sphinx to automatically convert markdown into comparable HTML. So, it's only the matter of adding it onto the file. I'm sure the documentation style is not hard to understand once you have had a chance to look into it. So, please do add it over there as well. And once merged the docs can be refreshed as well.

@hugoprudente
Copy link
Author

Cool. I'll work on the doc and send a new commit with it on the upcoming week. I'll be out-of-town for 1 weeks so Only when I get back be able to complete it.

Thanks.

@hugoprudente hugoprudente marked this pull request as draft July 28, 2023 18:57
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