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

Chart builder support for Trendline option in line chart #2167

Merged
merged 6 commits into from
Dec 4, 2024
Merged

Chart builder support for Trendline option in line chart #2167

merged 6 commits into from
Dec 4, 2024

Conversation

cnathe
Copy link
Contributor

@cnathe cnathe commented Dec 3, 2024

Rationale

See related PR for rationale. This PR adds test helpers for the chart wizard Trendline option UI components / panel.

Related Pull Requests

Changes

  • app chart helper for selecting new line chart Trendline option
  • add app grid helper for filtering a boolean column for true/false value
  • app chart helper for checking if Trendline option is available
  • LKS Chart wizard helper for trendline options in ChartTypeDialog

@cnathe cnathe mentioned this pull request Dec 3, 2024
Merged
@cnathe cnathe merged commit ff3597f into develop Dec 4, 2024
3 checks passed
@cnathe cnathe deleted the fb_lineChartTrendline branch December 4, 2024 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@labkey-danield labkey-danield labkey-danield approved these changes

@labkey-chrisj labkey-chrisj Awaiting requested review from labkey-chrisj

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

Successfully merging this pull request may close these issues.
2 participants
@cnathe @labkey-danield