-
Notifications
You must be signed in to change notification settings - Fork 62
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
Add sectorKey
and industryKey
to assetProfile
#689
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## devel #689 +/- ##
=======================================
Coverage 93.12% 93.12%
=======================================
Files 26 26
Lines 669 669
Branches 226 226
=======================================
Hits 623 623
Misses 46 46 ☔ View full report in Codecov by Sentry. |
Hey @martinnormark, thanks for another very welcome PR. Apologies for the delay as I've been travelling. Merging now and this will be in the next release. Thanks again for your time, efforts and patience. |
🎉 This PR is included in version 2.8.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Hey @gadicc - no worries, always great to contribute to open source. Hopefully I did the changes in the right way for schema validation. |
All looked perfect to me! 💯 Thanks for your attention to detail 😉 |
Fixed validation error in quote summary, caused by new fields in the
assetProfile
module.Changes
quoteSummary-iface.ts
and built schema.Type