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

Bug: Contributor table in a list crashes in card view when you change the page size to any value other than the current value #2240

Closed
2 tasks done
nickytonline opened this issue Nov 30, 2023 · 4 comments · Fixed by #2241
Labels
🐛 bug Something isn't working core team work Work that the OpenSauced core team takes on high-priority Work on these issues first released on @beta released

Comments

@nickytonline
Copy link
Member

Describe the bug

Contributor table in a list crashes when in card view when you change the page size to any value other than the current value.

Steps to reproduce

  1. For any list, go to the contributors tab.
  2. The grid loads fine.
  3. Switch to card view and the grid refreshes in card view.
  4. Change the page limit to any other value than the currently selected page size will cause the page to crash on the client-side with Next.js's client-side error page.

image

Browsers

No response

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@nickytonline nickytonline added 🐛 bug Something isn't working high-priority Work on these issues first core team work Work that the OpenSauced core team takes on labels Nov 30, 2023
Copy link
Contributor

Thanks for the issue, our team will look into it as soon as possible! If you would like to work on this issue, please wait for us to decide if it's ready. The issue will be ready to work on once we remove the "needs triage" label.

To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please reach out to us on Discord or follow up on the issue itself.

For full info on how to contribute, please check out our contributors guide.

@nickytonline
Copy link
Member Author

I dug into this an it appears to be an invalid updated_at field for a contributor. It's undefined which cause some date calculations to fail which causes the page to crash on the client-side with this error.

CleanShot 2023-11-30 at 18 41 40

Here's a video of me debugging the crash before it happens. Notice the updated_at is undefined along with other fields.

CleanShot.2023-11-30.at.18.37.10.mp4

Copy link
Contributor

open-sauced bot commented Dec 1, 2023

🎉 This issue has been resolved in version 1.78.1-beta.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link
Contributor

open-sauced bot commented Dec 5, 2023

🎉 This issue has been resolved in version 1.79.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@open-sauced open-sauced bot added the released label Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working core team work Work that the OpenSauced core team takes on high-priority Work on these issues first released on @beta released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant