Skip to content

Conversation

@rhino1998
Copy link
Contributor

Summary

Tweak the XKCD source plugin UI

  • Add logo
  • Change card title to "Configure source"
  • Add helper text to source field

@rhino1998 rhino1998 requested review from a team and blesniewski and removed request for a team August 2, 2024 15:17
Copy link
Contributor

@cduflo cduflo left a 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={
Copy link
Contributor

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rhino1998 rhino1998 added the automerge Automatically merge once required checks pass label Aug 2, 2024
@kodiakhq kodiakhq bot merged commit 5a20677 into main Aug 2, 2024
@kodiakhq kodiakhq bot deleted the feat/xkcd-source-cloud-plugin-ui-v2 branch August 2, 2024 15:51
kodiakhq bot pushed a commit that referenced this pull request Aug 5, 2024
🤖 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).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/plugin/source/xkcd automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants