Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ENH Replace griddle with Paginator component #391

Merged

Conversation

emteknetnz
Copy link
Member

Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

Compare mode doesn't work

  • Click the weird settings button in the top right
  • Check the "compare two versions" checkbox
  • Select any two versions (you can select across pages but you don't have to)
  • After selecting the second one, the comparison content doesn't load and an error is in the console

@emteknetnz
Copy link
Member Author

emteknetnz commented Mar 4, 2025

Works fine on my setup, including if I compare across paginated results i.e. one version of page one and another on page 2 (I've set SilverStripe\VersionedAdmin\Forms\HistoryViewerField.default_page_size to 3 via yml to get the pagination to show up)

Are you sure you also have the silverstripe/silverstripe-admin#1909 PR installed?

@GuySartorelli
Copy link
Member

I definitely had it installed - but pulling the latest changes from that PR has resolved whatever was going wrong.

@emteknetnz emteknetnz marked this pull request as ready for review March 4, 2025 23:13
@emteknetnz emteknetnz force-pushed the pulls/3.0/react-table branch from 52ba384 to 0a13837 Compare March 5, 2025 00:08
Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

LGTM

@GuySartorelli GuySartorelli merged commit 5ef6658 into silverstripe:3.0 Mar 5, 2025
8 of 13 checks passed
@GuySartorelli GuySartorelli deleted the pulls/3.0/react-table branch March 5, 2025 04:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants