Skip to content

Conversation

@developerkunal
Copy link

@developerkunal developerkunal commented Oct 20, 2023

In this commit, I've added the logic to calculate the total number of pages dynamically for the "My Cases" section on the dashboard. The total number of cases is now divided by the cases per page to determine the correct number of pages to display.

Fixes #1285


PR-Codex overview

Detailed summary

  • Added calculation for totalPages based on totalCases and casesPerPage.
  • Updated the totalPages prop in the Dashboard component to use the calculated value.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

In this commit, I've added the logic to calculate the total number of pages dynamically for the "My Cases" section on the dashboard. The total number of cases is now divided by the cases per page to determine the correct number of pages to display.

Fixes kleros#1285
@netlify
Copy link

netlify bot commented Oct 20, 2023

Deploy Preview for kleros-v2 ready!

Name Link
🔨 Latest commit a168713
🔍 Latest deploy log https://app.netlify.com/sites/kleros-v2/deploys/653ff9c68fa0910008c05d43
😎 Deploy Preview https://deploy-preview-1292--kleros-v2.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 site configuration.

@developerkunal
Copy link
Author

developerkunal commented Oct 24, 2023

@jaybuidl can you check the PR

@jaybuidl
Copy link
Member

@jaybuidl can you check the PR

Seems the dashboard page crashes: https://deploy-preview-1292--kleros-v2.netlify.app/#/dashboard/1/desc/all

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

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit a168713 and detected 0 issues on this pull request.

View more on Code Climate.

@kemuru
Copy link
Contributor

kemuru commented Nov 16, 2023

solved by #1332

@kemuru kemuru closed this Nov 16, 2023
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.

Dashboard, My Cases: pagination overflows

3 participants