Skip to content

Add support for global synonym/curation sets (Typesense v30+)#304

Open
WimVergouwe wants to merge 5 commits intoDAXGRID:masterfrom
WimVergouwe:feature/typesense-v30
Open

Add support for global synonym/curation sets (Typesense v30+)#304
WimVergouwe wants to merge 5 commits intoDAXGRID:masterfrom
WimVergouwe:feature/typesense-v30

Conversation

@WimVergouwe
Copy link
Contributor

  • Add global synonym set and curation set APIs to ITypesenseClient
  • Add new record types for synonym/curation set schemas and responses
  • Update CollectionResponse and Schema for synonym_sets/curation_sets
  • Add compatibility/version logic to Config and TypesenseClient
  • Enforce feature support at runtime with NotSupportedException
  • Update and expand tests for new APIs, using SkippableFact
  • Allow test fixture to configure minimum compatibility version
  • Miscellaneous test and error handling improvements

- Add global synonym set and curation set APIs to ITypesenseClient
- Add new record types for synonym/curation set schemas and responses
- Update CollectionResponse and Schema for synonym_sets/curation_sets
- Add compatibility/version logic to Config and TypesenseClient
- Enforce feature support at runtime with NotSupportedException
- Update and expand tests for new APIs, using SkippableFact
- Allow test fixture to configure minimum compatibility version
- Miscellaneous test and error handling improvements
@runeanielsen
Copy link
Member

Hi @WimVergouwe,

I am currently a bit busy IRL, but hopefully I can do a review soon.

Thanks,
Rune Nielsen

@runeanielsen
Copy link
Member

Hi @WimVergouwe,

Thank you for the contribution.

I've reviewed a few things so far, and I think it looks good. I think I would rather we do not introduce the version check, and I'll jump bump it to be a new major version. I think the complexity added does not add enough benefit.

If you can remove that I'll take another look after.

Best regards,
Rune

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.

3 participants