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
Bug fix: Warns user if incorrect SDK version is installed #4422
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution! Can you add a changelog fragment? Thanks.
This comment was marked as outdated.
This comment was marked as outdated.
Added. Also changed the bare exception as suggested. Thanks for the review. |
changelogs/fragments/4422-warn-user-if-incorrect-SDK-version-is-installed.yaml
Outdated
Show resolved
Hide resolved
Co-authored-by: Felix Fontein <felix@fontein.de>
Backport to stable-3: 💚 backport PR created✅ Backport PR branch: Backported as #4449 🤖 @patchback |
* Add error handling to check correct SDK version installed * Fix CI errors * Added changelog fragment * Changed exeption type * Update changelogs fragment Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit e7ffa76)
@EndlessTrax thanks for fixing this! |
Backport to stable-4: 💚 backport PR created✅ Backport PR branch: Backported as #4450 🤖 @patchback |
* Add error handling to check correct SDK version installed * Fix CI errors * Added changelog fragment * Changed exeption type * Update changelogs fragment Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit e7ffa76)
) * Add error handling to check correct SDK version installed * Fix CI errors * Added changelog fragment * Changed exeption type * Update changelogs fragment Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit e7ffa76) Co-authored-by: Ricky White <rickywhite@outlook.com>
) * Add error handling to check correct SDK version installed * Fix CI errors * Added changelog fragment * Changed exeption type * Update changelogs fragment Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit e7ffa76) Co-authored-by: Ricky White <rickywhite@outlook.com>
SUMMARY
This raises an error if the wrong
python-dsv-sdk
version is installed. This lookup plugin currently only supports v0.0.1 of the SDK. This is a temporary fix until full support with the latest SDK version can be developed.(Re-submitted with clean git history)
ISSUE TYPE
COMPONENT NAME
dsv lookup plugin
ADDITIONAL INFORMATION
Before the fix, the error presented to the user:
After: