You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently only a single possible index is support, due to the global variable index in adapter.ts. It would be helpful if getSearchClient could take an index parameter. It currently looks like it does, since it takes the parameternewIndex?: any, but this newIndex is only used as index || newIndex, so if there's an existing index, newIndex ignored. Not sure if that's intentional. Hope my issue is clear. Thanks for developing and maintaining this library.
The text was updated successfully, but these errors were encountered:
Hi @chammaaomar thank you for using the package. As it has been a while since I last worked on the library, I am not sure if it was done intentional or not. I currently do not have the time to test or make any updates. I suggest that you consider implementing support for multiple indexes yourself and submitting a pull request for review. This would be the best way forward.
Currently only a single possible index is support, due to the global variable
index
in adapter.ts. It would be helpful ifgetSearchClient
could take an index parameter. It currently looks like it does, since it takes the parameternewIndex?: any
, but thisnewIndex
is only used asindex || newIndex
, so if there's an existing index,newIndex
ignored. Not sure if that's intentional. Hope my issue is clear. Thanks for developing and maintaining this library.The text was updated successfully, but these errors were encountered: