Skip to content
This repository has been archived by the owner on Sep 1, 2024. It is now read-only.

Dynamic charts #116

Merged
merged 8 commits into from
Feb 7, 2024
Merged

Dynamic charts #116

merged 8 commits into from
Feb 7, 2024

Conversation

adibar121
Copy link
Collaborator

  • create dynamic chart component
  • add selector for chart type, xaxios , yaxios
  • set chart by selector selections

E4DC0CAE-381D-4151-857F-749E89F56A4A

Checklist:

  • I have included a meaningful summary of my PR's content
  • If applicable, I have included screenshots to help describe my change
  • I have linked my PR to the relevant issue
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes do not generate new compliance warnings / errors
  • I have tested that my fix or enhancement is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have made corresponding changes to the documentation

@adibar121 adibar121 requested a review from ohadkoren February 4, 2024 07:40
@adibar121 adibar121 self-assigned this Feb 4, 2024
@ohadkoren ohadkoren added this to the 1.2.0 milestone Feb 7, 2024
@ohadkoren ohadkoren linked an issue Feb 7, 2024 that may be closed by this pull request
[Experiment Page] Dynamic charts #98
Closed
@ohadkoren ohadkoren added the task New task label Feb 7, 2024
@adibar121 adibar121 merged commit 885b200 into main Feb 7, 2024
4 checks passed
@adibar121 adibar121 deleted the dynamic_charts branch February 7, 2024 09:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ohadkoren ohadkoren ohadkoren approved these changes

Assignees

@adibar121 adibar121

Labels
task New task
Projects
Qujata
Status: Done
Milestone
1.2.0
Development

Successfully merging this pull request may close these issues.

[Experiment Page] Dynamic charts
3 participants
@adibar121 @ohadkoren @iadibar