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

Search functionality in flowershow.app website #34

Closed
7 tasks done
rufuspollock opened this issue Jul 14, 2022 · 6 comments
Closed
7 tasks done

Search functionality in flowershow.app website #34

rufuspollock opened this issue Jul 14, 2022 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@rufuspollock
Copy link
Member

rufuspollock commented Jul 14, 2022

When on flowershow website i want to quickly search for information

Acceptance

  • Search in navbar a la tailwindcss.com etc ✅ PR Search functionality feature on website #81
  • Docs for this feature ✅ done /docs/search
    • user section explaining what search is and maybe short screenshot.
    • admin section explaining how to configure (e.g. what fields need to be set where)

Tasks

  • Register on algolia docsearch ✅2022-08-08 did this and have stuff for flowershow.app
  • Implement Search component and use
  • Docs
@olayway olayway added the enhancement New feature or request label Jul 14, 2022
@rufuspollock rufuspollock changed the title Search on app website Search functionality in flowershow.app website Jul 18, 2022
@rufuspollock
Copy link
Member Author

@olayway can we submit now to algolia docsearch as can take a bit of time to get accepted.

@rufuspollock
Copy link
Member Author

@olayway i've taken care of submitting this so don't worry.

@khalilcodes
Copy link
Contributor

@rufuspollock config requirements for implementing search in algolia - App ID, Api key, Index name.

This is how it looks as per current PR status - #78

@rufuspollock
Copy link
Member Author

@olayway do you want to review this one? You also have the config info we need here - i'm not sure all of it can be public (?). If not we may need to pull some from env variables.

@rufuspollock
Copy link
Member Author

@olayway i've gone ahead and reviewed quickly.

@olayway
Copy link
Member

olayway commented Aug 10, 2022

Done in PR #81

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

No branches or pull requests

3 participants