Skip to content
This repository was archived by the owner on Sep 3, 2025. It is now read-only.

Conversation

@mvilanova
Copy link
Contributor

This pull request includes several changes to the Page.vue and store.js files in the src/dispatch/static/dispatch/src/case directory. These changes primarily focus on adding support for displaying GenAI analysis, reordering properties for better readability, and importing necessary modules.

Enhancements to GenAI Analysis Support:

  • Added GenaiAnalysisDisplay component to the Page.vue template for displaying GenAI analysis when the active tab is not 'signals' or 'graph'. (src/dispatch/static/dispatch/src/case/Page.vue)
  • Included genai_analysis property in the caseDefaults object and updated the PageHeader component to accept this new property. (src/dispatch/static/dispatch/src/case/Page.vue) [1] [2]

Code Reorganization:

  • Reordered properties in the caseDefaults object and the getDefaultSelectedState function for better readability and consistency. (src/dispatch/static/dispatch/src/case/Page.vue, src/dispatch/static/dispatch/src/case/store.js) [1] [2] [3] [4] [5]

Module Imports:

  • Adjusted the import statements in Page.vue to ensure all necessary modules are properly imported and organized. (src/dispatch/static/dispatch/src/case/Page.vue)
Screenshot 2024-10-08 at 5 04 42 PM Screenshot 2024-10-08 at 5 04 29 PM

@mvilanova mvilanova added the enhancement New feature or request label Oct 9, 2024
@wssheldon
Copy link
Contributor

wssheldon commented Oct 9, 2024

I can make the change in a follow up PR. For the design, I'd suggest to change the Timeline tab to Overview and move the GenAI summary into a Card that is displayed side by side with the timeline

@mvilanova mvilanova merged commit e825b87 into master Oct 9, 2024
@mvilanova mvilanova deleted the enhancement/display-genai-analysis-ui branch October 9, 2024 22:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants