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.

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

Implement report feature suggestions #1009

Merged
merged 5 commits into from
Sep 30, 2024
Merged

Conversation

hvangeffen
Copy link
Collaborator

@hvangeffen hvangeffen commented Sep 26, 2024

Description

  • Now uses moduleInstanceName as report name
  • Reduce font size of the report title
  • Add report download button

Downloaded file name example: 2024-09-26T10_45_00Z-ReportRainfallReturnPeriodTableForecast.htm

Also excludes vuetify from optimizedDeps since with the treeshake change it has to reload the vite server every time a new vuetify component is used within a newly loaded async component.

Screenshots

image

Checklist

  • Make the title short and concise
  • Select the correct label to include it in the release notes:
    • rel: new feature
    • rel: improvement
    • rel: fixes
    • rel: ignore
      Select to rel: ignore if this pull request fixes a New Feature or Improvement in the coming release. Update related Pull Request.
  • Update documentation.
  • Update tests.

@hvangeffen hvangeffen linked an issue Sep 26, 2024 that may be closed by this pull request
vite.config.ts Outdated
@@ -74,7 +74,7 @@ export default defineConfig(({ mode }) => {
}),
],
optimizeDeps: {
exclude: ['@deltares/fews-ssd-webcomponent'],
exclude: ['@deltares/fews-ssd-webcomponent', 'vuetify'],
Copy link
Collaborator

Choose a reason for hiding this comment

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

Put in its own pull request

@wkramer wkramer force-pushed the 1008-implement-report-suggestions branch from 7ca7c74 to 3b61a4c Compare September 30, 2024 07:04
@wkramer wkramer merged commit 902cf2c into main Sep 30, 2024
8 checks passed
@wkramer wkramer deleted the 1008-implement-report-suggestions branch September 30, 2024 07:11
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.

Implement report suggestions
2 participants