Skip to content

Conversation

martedesco
Copy link
Collaborator

This pull request includes changes to the GitHub Copilot metrics viewer application. The changes are focused on improving the code structure, updating the application version, and removing an old and not used the Azure Static Web Apps CI/CD workflow. The most important changes include updating the application version and refactoring the Vue components to improve the code structure and readability.

Removal of Azure Static Web Apps CI/CD workflow:

Package version upgrade:

  • package.json: The version of copilot-metrics-viewer has been updated from 0.1.0 to 1.2.0. This indicates the introduction of new features or significant changes.

Addition of new component:

Major modifications to existing components:

  • src/components/CopilotChatViewer.vue: This component underwent significant changes, including the removal of error handling code, the addition of props for passing metrics data, changes to the setup function to use the passed props, and changes to the chart data generation code to use the passed props. [1] [2] [3] [4] [5] [6] [7]

  • src/components/LanguagesBreakdown.vue: Similar to CopilotChatViewer.vue, this component also had significant changes, including the removal of error handling code, the addition of props for passing metrics data, changes to the setup function to use the passed props, and changes to the chart data generation code to use the passed props. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

  • src/components/MainComponent.vue: This component was updated to handle the API call and pass the metrics data to the child components. It also handles API errors and displays a progress bar until the metrics data is ready. [1] [2]

@martedesco martedesco merged commit e32566c into main Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant