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.

Sign up for GitHub

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

Jump to bottom

Add event handler on Timeseries #366

Merged
merged 5 commits into from
Sep 28, 2023
Merged

Add event handler on Timeseries #366

merged 5 commits into from
Sep 28, 2023

Conversation

Finz-Azmi
Copy link
Contributor

This PR

Closes #xxx

Notes

  • No notes to add

Fulfilled the scope of this repo?

  • The same functionality of the component can not be achieved with theming, basic styling or composition of existing components
  • The component implements an element of the DHI Design Guidelines or is otherwise generic enough in functionality and close enough to the DHI CVI that it is likely to find reuse in other projects

Completness

  • Story included

FranzThomsen1089
FranzThomsen1089 approved these changes Sep 28, 2023
View reviewed changes
Copy link
Collaborator

@FranzThomsen1089 FranzThomsen1089 left a comment

Choose a reason for hiding this comment

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

All good

@FranzThomsen1089 FranzThomsen1089 merged commit 67b00aa into master Sep 28, 2023
@FranzThomsen1089 FranzThomsen1089 deleted the feature/timeseries-event-handler branch September 28, 2023 04:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FranzThomsen1089 FranzThomsen1089 FranzThomsen1089 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Finz-Azmi @FranzThomsen1089 @zamzfz