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

Feature/azure search Help #303

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from

Conversation

ArgTang
Copy link
Contributor

@ArgTang ArgTang commented Sep 23, 2018

Hi
Call for help: Azure search throws on Article.Instant because it is a nodatime property.

I have been trying to implement pluggable search with Azure search as another provider. I have hit a wall with Azure search not wanting to serialise Nodatimes Instant type. If Anyone have worked with Azure search, or know who i can ask, please let me know.

Also first time for many of the patterns used here, if you have any feedback on those I'd love to hear it

Note: this is very much work in progress.

To reproduce:

  • create a free tier azure search in azure portal.
  • Insert appropriate keys in appsettings.json.
  • change SearchProvider to "Az"
  • Edit an article to trigger crash in AzureSearchClient: line46

@csharpfritz
Copy link
Owner

I like the initiative here... I have a guest joining me on Friday’s show that will help kick off a “CoreWiki.Azure” project that we can move your search capabilities into.

@ArgTang
Copy link
Contributor Author

ArgTang commented Sep 24, 2018

I've been watching some Vods and got inspired try try to make something with some patterns in it. Also by making search pluggable, elastic integration would be up for grabs as well.

If this is in a state that you can use it as a base for a workshop i'm happy :)
Sounds cool, i'll have to try too catch the stream and see the endresult

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