Skip to content

Conversation

@andoriyaprashant
Copy link
Contributor

What does this PR change?

  • Updates deprecated props onChangePage and onChangeRowsPerPage to onPageChange and onRowsPerPageChange in TablePagination components.
  • Fixes missing required prop onPageChange that caused console warnings.

Does this PR relate to any other PRs?

  • No direct relation to other PRs.

How will this PR impact users?

  • Eliminates console warning errors related to TablePagination usage, improving developer experience and code compatibility with Material UI v5.

Does this PR address any GitHub or Zendesk issues?

How was this PR tested?

  • Manually debugged and ran the UI components in allocationReport.js, cloudCost.js, and externalCostsTable.js to verify pagination works correctly without console errors.

Does this PR require changes to documentation?

  • No

@netlify
Copy link

netlify bot commented May 16, 2025

Deploy Preview for opencost-ui ready!

Name Link
🔨 Latest commit 56121e8
🔍 Latest deploy log https://app.netlify.com/projects/opencost-ui/deploys/6838c0923845bb00082f3d1f
😎 Deploy Preview https://deploy-preview-84--opencost-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: andoriyaprashant <prashantandoriya@gmail.com>
@lmello
Copy link
Member

lmello commented May 29, 2025

thank you for your contribution.

@ameijer ameijer merged commit 244d62b into opencost:main Jul 10, 2025
6 checks passed
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.

Update deprecated and missing props in TablePagination component

3 participants