-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: Add new components to simplify migration from DataSourceHttpSettings #67
feat: Add new components to simplify migration from DataSourceHttpSettings #67
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.
After copying over the new components to core in order to test, this seems to work well 👍
Note: I didn't review the code too much, I guess it's more or less the same as it was before you extracted it with maybe a few minor improvements. I'll leave that part to you and @sasklacz
Documentation is clear and concise.
Changes
ConnectionSettings
andAdvancedHttpSettings
components to simplify migration fromDataSourceHttpSettings
component.Screenshots of new components