-
Notifications
You must be signed in to change notification settings - Fork 547
feat: XKCD source plugin UI v2 #18837
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
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.
nothing blocking, 2 small suggestions
nice!! 🚀
| error={!!fieldState.error} | ||
| fullWidth={true} | ||
| helperText={fieldState.error?.message} | ||
| helperText={ |
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.
@rhino1998 we have a new pattern (very recent, you didn't miss it) to use the getFieldHelperText util from the plugin-config-ui-lib that will keep showing the helper text in addition to the error text
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.
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.
so minor, but may be worth converting to webp
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.
🤖 I have created a release *beep* *boop* --- ## [1.3.0](plugins-source-xkcd-v1.2.0...plugins-source-xkcd-v1.3.0) (2024-08-04) ### Features * XKCD source plugin UI v2 ([#18837](#18837)) ([5a20677](5a20677)) ### Bug Fixes * **deps:** Update dependency @cloudquery/cloud-ui to v0.1.12 ([#18836](#18836)) ([b24afab](b24afab)) * **deps:** Update dependency @cloudquery/plugin-config-ui-connector to v0.2.10 ([#18850](#18850)) ([ec926b5](ec926b5)) * **deps:** Update dependency @cloudquery/plugin-config-ui-lib to ^0.0.42 ([#18834](#18834)) ([827d13b](827d13b)) * **deps:** Update dependency @cloudquery/plugin-config-ui-lib to ^0.0.44 ([#18838](#18838)) ([745aeca](745aeca)) * **deps:** Update dependency @cloudquery/plugin-config-ui-lib to ^0.0.45 ([#18841](#18841)) ([4476f38](4476f38)) * **deps:** Update dependency @cloudquery/plugin-config-ui-lib to ^0.0.46 ([#18844](#18844)) ([60f5924](60f5924)) * **deps:** Update dependency @cloudquery/plugin-config-ui-lib to ^0.0.47 ([#18849](#18849)) ([029638e](029638e)) * **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.57.1 ([#18830](#18830)) ([605c202](605c202)) * **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.58.0 ([#18839](#18839)) ([6b57bca](6b57bca)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Summary
Tweak the XKCD source plugin UI