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

Move Report Viewer to Composition API #1034

Merged
merged 3 commits into from
Apr 20, 2023
Merged

Conversation

Kr0nox
Copy link
Member

@Kr0nox Kr0nox commented Apr 19, 2023

This PR changes the report viewer from a mix of Vues Options and Composition API to being solely written in the Composition API

@Kr0nox Kr0nox requested review from sebinside and TwoOfTwelve April 19, 2023 17:15
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@tsaglam tsaglam added enhancement Issue/PR that involves features, improvements and other changes minor Minor issue/feature/contribution/change report-viewer PR / Issue deals (partly) with the report viewer and thus involves web-dev technologies labels Apr 20, 2023
@sebinside sebinside removed the request for review from TwoOfTwelve April 20, 2023 14:23
@sebinside sebinside merged commit 963bd9d into develop Apr 20, 2023
@sebinside sebinside deleted the report-viewer/composition-api branch April 20, 2023 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue/PR that involves features, improvements and other changes minor Minor issue/feature/contribution/change report-viewer PR / Issue deals (partly) with the report viewer and thus involves web-dev technologies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants