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

[GLT-4261] TAT trend report table metrics #232

Merged
merged 5 commits into from
Nov 15, 2024
Merged

Conversation

wuall826
Copy link
Contributor

@wuall826 wuall826 commented Oct 11, 2024

Jira ticket or GitHub issue:

  • Includes a change file
  • Updated tests (or n/a)
  • Updated documentation (or n/a)
  • Discussed with CGI (or n/a; see release procedure on wiki for applicability)

@wuall826
Copy link
Contributor Author

wuall826 commented Oct 11, 2024

still need to stick "Assay" and "Gate" column, will add another commit. also, would it be better if I used the backend java csv download (like for other reports) for the metric download in this?

@wuall826 wuall826 requested a review from djcooke October 11, 2024 17:33
@wuall826 wuall826 marked this pull request as draft October 11, 2024 17:34
@wuall826 wuall826 force-pushed the GLT-4261_tableMetrics branch from 1175170 to 0f6220a Compare October 11, 2024 20:39
src/main/resources/templates/tat-trend.html Outdated Show resolved Hide resolved
ts/component/table-builder.ts Outdated Show resolved Hide resolved
ts/component/table-builder.ts Outdated Show resolved Hide resolved
ts/component/table-builder.ts Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
@wuall826 wuall826 marked this pull request as ready for review October 31, 2024 23:11
@wuall826 wuall826 requested review from djcooke and avarsava October 31, 2024 23:11
ts/component/table-builder.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
@wuall826 wuall826 requested a review from djcooke November 6, 2024 19:16
ts/tat-trend.ts Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
ts/component/table-builder.ts Outdated Show resolved Hide resolved
ts/tat-trend.ts Outdated Show resolved Hide resolved
@wuall826 wuall826 merged commit b47d32c into main Nov 15, 2024
5 checks passed
@wuall826 wuall826 deleted the GLT-4261_tableMetrics branch November 15, 2024 14:02
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