Skip to content

Typesense updates#1

Open
shivag46 wants to merge 20 commits into
masterfrom
typesense-updates
Open

Typesense updates#1
shivag46 wants to merge 20 commits into
masterfrom
typesense-updates

Conversation

@shivag46

@shivag46 shivag46 commented Apr 5, 2021

Copy link
Copy Markdown

Change Summary

PR Checklist

@jasonbosco

Copy link
Copy Markdown
Member

@shivag46 At a high level the approach looks good.

However, it looks like there a couple files which have bad merge code. For eg. Could you resolves these merge issues?

Separately, I also see some code that's commented out like this. I'm guessing it's just to test functionality out?

Could you rename includes/libraries/typesensesearch-client-php to includes/libraries/typesense-php since that's the name of the library?

@shivag46

shivag46 commented Apr 7, 2021

Copy link
Copy Markdown
Author

I have resolved the merge conflicts and renamed the library folder.

And about the commented out sections, few of them were Algolia related code that were unnecessary for our project and the rest are some additional features that I need to work on.

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