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

feat(pci-databases-analytics) - refactor data table and add filtering #14524

Merged
merged 3 commits into from
Jan 17, 2025

Conversation

jperchoc
Copy link
Contributor

@jperchoc jperchoc commented Dec 10, 2024

Question Answer
Branch? develop
Bug fix? no
New feature? yes
Breaking change? no
Tickets Fix #DATATR-1763
License BSD 3-Clause
  • Try to keep pull requests small so they can be easily reviewed.
  • Commits are signed-off
  • Only FR translations have been updated
  • Branch is up-to-date with target branch
  • Lint has passed locally
  • Standalone app was ran and tested locally
  • Ticket reference is mentioned in linked commits (internal only)
  • Breaking change is mentioned in relevant commits

Description

Update data-table to add sorting and filtering

Related

The branch feat/mysql-query-statistics should be merged into this branch before proceeding to the release

Base automatically changed from feat/pci-db-analytics-rework-modals to release/public-cloud-w51 December 12, 2024 16:40
@github-actions github-actions bot added the has conflicts Has conflicts to resolve before merging label Dec 12, 2024
@fredericvilcot fredericvilcot force-pushed the release/public-cloud-w51 branch 2 times, most recently from 6b29873 to afc8dfc Compare December 12, 2024 17:02
@jperchoc jperchoc force-pushed the feat/data-table-sort-filter branch 2 times, most recently from 9a88aaf to de328be Compare December 13, 2024 16:02
@github-actions github-actions bot removed the has conflicts Has conflicts to resolve before merging label Dec 13, 2024
@jperchoc jperchoc changed the title Feat/data table sort filter feat(pci-databases-analytics) - refactor data table and add filtering Dec 16, 2024
@fredericvilcot fredericvilcot force-pushed the release/public-cloud-w51 branch from 54e17af to 9da7f4f Compare December 16, 2024 13:21
@github-actions github-actions bot added has conflicts Has conflicts to resolve before merging universe-public-cloud labels Dec 16, 2024
@fredericvilcot fredericvilcot force-pushed the release/public-cloud-w51 branch 3 times, most recently from 58793cd to 0279f9d Compare December 16, 2024 13:58
Base automatically changed from release/public-cloud-w51 to master December 16, 2024 14:01
@jperchoc jperchoc force-pushed the feat/data-table-sort-filter branch from 878f3d5 to 4c1e100 Compare December 16, 2024 14:07
@github-actions github-actions bot added translation required and removed universe-public-cloud has conflicts Has conflicts to resolve before merging labels Dec 16, 2024
@jperchoc jperchoc marked this pull request as ready for review December 16, 2024 14:11
@jperchoc jperchoc requested review from a team as code owners December 16, 2024 14:11
Signed-off-by: Jonathan Perchoc <[email protected]>
@jperchoc jperchoc force-pushed the feat/data-table-sort-filter branch from ccfdef7 to e3dff38 Compare December 26, 2024 13:23
abullet33
abullet33 previously approved these changes Dec 30, 2024
Comment on lines +3 to +5
export function DatatableAction({ children }: { children: ReactNode }) {
return <>{children || <></>}</>;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
export function DatatableAction({ children }: { children: ReactNode }) {
return <>{children || <></>}</>;
}
export function DatatableAction({ children }: { children: ReactNode }) {
return <>{children || undefined}</>;
}

tibs245
tibs245 previously approved these changes Jan 9, 2025
Copy link
Contributor

@tibs245 tibs245 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not forget to squash :)

ref: DATATR-681

Signed-off-by: Jonathan Perchoc <[email protected]>
@jperchoc jperchoc dismissed stale reviews from tibs245 and abullet33 via b099c46 January 16, 2025 11:02
@jperchoc jperchoc changed the base branch from master to develop January 16, 2025 11:03
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
5.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@acavalloni acavalloni requested a review from tibs245 January 16, 2025 13:45
@anooparveti anooparveti changed the base branch from develop to release/public-cloud-w4 January 17, 2025 06:56
@anooparveti
Copy link
Contributor

Failed: Install Playwright Browsers

The error is ignored as the error has nothing to do with the changes in PR and the error was resolved with #14959

@anooparveti anooparveti merged commit 18949ed into release/public-cloud-w4 Jan 17, 2025
16 of 19 checks passed
@anooparveti anooparveti deleted the feat/data-table-sort-filter branch January 17, 2025 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants