Skip to content
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 'Color-Coded Heatmap' widget #4629

Merged

Conversation

yuda110
Copy link
Member

@yuda110 yuda110 commented Sep 2, 2024

To Reviewers

  • Skip (style, chore, ci, minor refactoring, etc.)
  • Need discussion
  • Not that difficult
  • Approved feature branch merge to master

Description

스크린샷 2024-09-02 오후 4 40 08

Copy link
Member

@piggggggggy piggggggggy left a comment

Choose a reason for hiding this comment

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

LGTM~ I left a small comment

state.proxyValue = {
...state.proxyValue,
value: state.proxyValue?.value ?? [],
baseColor: state.proxyValue?.baseColor ?? props.widgetFieldSchema?.options?.baseColor ?? DEFAULT_BASE_COLOR,
Copy link
Member

@piggggggggy piggggggggy Sep 2, 2024

Choose a reason for hiding this comment

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

I think.. baseColor: props.value?.baseColor... is better than baseColor: state.proxyValue?.baseColor....
Maybe not working.

@yuda110 yuda110 merged commit 0f3fe6c into cloudforet-io:feature-dashboard-september Sep 2, 2024
2 checks passed
@yuda110 yuda110 deleted the feature/color-coded branch September 2, 2024 08:12
yuda110 added a commit to yuda110/cloudforet-console that referenced this pull request Sep 3, 2024
* feat: add `Color-Coded Heatmap` widget

Signed-off-by: yuda <[email protected]>

* feat: add to widget config list

Signed-off-by: yuda <[email protected]>

* fix(advanced-format-rules): set init value when mounted

Signed-off-by: yuda <[email protected]>

* chore: update language

Signed-off-by: yuda <[email protected]>

* chore: delete unused variable

Signed-off-by: yuda <[email protected]>

* chore: add icon

Signed-off-by: yuda <[email protected]>

* chore: apply code review

Signed-off-by: yuda <[email protected]>

---------

Signed-off-by: yuda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants