Skip to content

Conversation

@edgardavid2015
Copy link
Contributor

Problem

Currently in the Reports section, when the CSV or Excel buttons are selected, these do not show the ID columns in the exported file.

problem_with_ID_columns

Solution

The previous functionality of the buttons was implemented.

previos_functionality

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 2, 2021

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
0.0% 0.0% Duplication

@codecov
Copy link

codecov bot commented Sep 2, 2021

Codecov Report

Merging #745 (6192745) into master (2b16e33) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #745   +/-   ##
=======================================
  Coverage   95.54%   95.54%           
=======================================
  Files         106      106           
  Lines        2247     2247           
  Branches      169      169           
=======================================
  Hits         2147     2147           
  Misses         57       57           
  Partials       43       43           
Impacted Files Coverage Δ
...time-entries-table/time-entries-table.component.ts 77.77% <ø> (ø)
...les/shared/components/sidebar/sidebar.component.ts 90.47% <ø> (ø)
...e-toggle-general/feature-toggle-general.service.ts 100.00% <ø> (ø)
.../feature-toggles/feature-toggle-manager.service.ts 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0675eef...6192745. Read the comment docs.

@edgardavid2015 edgardavid2015 merged commit 1ec686a into master Sep 2, 2021
@edgardavid2015 edgardavid2015 deleted the TT-333-Fix-CSV-Excel-buttons branch September 2, 2021 16:00
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.

3 participants