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

Default options - chart #134

Merged
merged 3 commits into from
Feb 18, 2024
Merged

Default options - chart #134

merged 3 commits into from
Feb 18, 2024

Conversation

adibar121
Copy link
Collaborator

  • add default options to the chart

799C5C1A-006F-44D8-A066-D456060FBFE2

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 self-assigned this Feb 18, 2024
@adibar121 adibar121 merged commit e5f1354 into main Feb 18, 2024
4 checks passed
@adibar121 adibar121 deleted the default_charts branch February 18, 2024 13:34
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
None yet
Projects
Qujata
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adibar121 @ohadkoren @iadibar