Skip to content

LIMS: Charting Story #3 - audit logs, cross folder & rendering improvements#1597

Merged
cnathe merged 28 commits intodevelopfrom
fb_charts2410
Oct 4, 2024
Merged

LIMS: Charting Story #3 - audit logs, cross folder & rendering improvements#1597
cnathe merged 28 commits intodevelopfrom
fb_charts2410

Conversation

@cnathe
Copy link
Copy Markdown
Contributor

@cnathe cnathe commented Sep 30, 2024

Rationale

This PR allows for Generic Chart reports to be inherited to child folders, fixes the containerFilter used for chart getQuery requests, and improves some rendering issues. This also removes the experimental feature flag for the LIMS Chart Builder.

Related Pull Requests

Changes

  • updates to chart margins and sizes for chart builder and dashboard
  • only show 'x' grid lines for bar and box plots
  • use getContainerFilterForFolder() for chart getQuery request
  • Show "inheritable" checkbox in chart builder for users with proper perm (and if in app home)
  • remove Chart Builder experimental feature flag

  - updates to chart margins and sizes for chart builder and dashboard
  - only show 'x' grid lines for bar and box plots
# Conflicts:
#	packages/components/package-lock.json
#	packages/components/package.json
#	packages/components/releaseNotes/components.md
# Conflicts:
#	packages/components/package-lock.json
#	packages/components/package.json
#	packages/components/releaseNotes/components.md
cnathe added 5 commits October 1, 2024 08:05
# Conflicts:
#	packages/components/package-lock.json
#	packages/components/package.json
#	packages/components/releaseNotes/components.md
cnathe added 4 commits October 2, 2024 08:27
# Conflicts:
#	packages/components/package-lock.json
#	packages/components/package.json
#	packages/components/releaseNotes/components.md
@cnathe cnathe merged commit cf4edb0 into develop Oct 4, 2024
@cnathe cnathe deleted the fb_charts2410 branch October 4, 2024 20:25
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