[DataGrid] Save paging state in URL

[svrooij]

Fixed #2839

Pull Request

📖 Description

Paging or sorting for a data grid will now save the state in the url, if the developer opt-ed in for this on that grid. It also supports a prefix, in case you would have multiple grids on a single page.

🎫 Issues

• feat: add url support to DataGrid #2839

👩‍💻 Reviewer Notes

📑 Test Plan

✅ Checklist

General

• I have added tests for my changes.
• I have tested my changes.
• I have updated the project documentation to reflect my changes.
• I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

• I have added a new component
• I have added Unit Tests for my new component
• I have modified an existing component
• I have validated the Unit Tests for an existing component 404

⏭ Next Steps

[svrooij]

@vnbaaij and @dvoituron the tests work on my machine, I see the build is also failing on the dev branch. Is that something that needs investigating?

For one of my projects I use the GitHub Actions test logger to produce these nice looking stats in the action. It will also show exactly which test failed (if any). Is that something you might also want in your pipeline?

[dvoituron]

@vnbaaij and @dvoituron the tests work on my machine, I see the build is also failing on the dev branch. Is that something that needs investigating?

We have a similar process in our Pipeline:

But the problem comes with user rights (for automatic publication in the RP comment stream): Repo owners see this report but not you :-)

But in this case, it's more a .NET9 issue. @vnbaaij did you changed something?

[svrooij]

One log line said it could not download dotnet 9.0 ga, so I think it has to do with the official release of dotnet9

[vnbaaij]

One log line said it could not download dotnet 9.0 ga, so I think it has to do with the official release of dotnet9

There was a reference still to .net9 preview in a pipeline. This has been corrected

But in this case, it's more a .NET9 issue. @vnbaaij did you changed something?

Yes, but this test has already been fixed

[svrooij]

@vnbaaij someone has to press the button to run the tests for the final time.