-
Notifications
You must be signed in to change notification settings - Fork 10
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
fix: Handle missing decorator for model #364
Conversation
Thank you @tachyonicbytes for this first contribution! |
Reviewpad Report
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #364 +/- ##
==========================================
+ Coverage 90.24% 90.26% +0.02%
==========================================
Files 29 29
Lines 5595 5608 +13
Branches 657 661 +4
==========================================
+ Hits 5049 5062 +13
Misses 545 545
Partials 1 1
☔ View full report in Codecov by Sentry. |
Can you please fix the PR title. (Suffixing it with "fix:" should be enough)
Sure |
🎉 This PR is included in version 1.1.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
What type of PR is this? (check all applicable)
Description
Solves #343. Return a more descriptive error message when the
@TigrisSearchIndex
decorator is not set.Related Tickets & Documents
Related Issue Improve error messaging when a createOrUpdateIndex is called but the TigrisSearchIndex for the index hasn't been set #343
Closes Improve error messaging when a createOrUpdateIndex is called but the TigrisSearchIndex for the index hasn't been set #343
Added/updated tests?
have not been included
Is this change backwards compatible?
Does it require updates to Tigris docs?
and replace this text as
tigrisdata/tigris-docs#123
Checklist
npm run build
- builds successfullynpm run test
- tests passingnpm run lint
- no lint errors[optional] Are there any post deployment tasks we need to perform?
/claim #343