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

Support sorting from GridProxy in Contracts List page #3171

Merged
merged 3 commits into from
Jul 29, 2024

Conversation

Mahmoud-Emad
Copy link
Contributor

@Mahmoud-Emad Mahmoud-Emad commented Jul 28, 2024

Description

Support sorting from GridProxy on the Contracts List page

Changes

  • Updated the GridProxy client to include SortBy and SortOrder fields
  • Created a new enum to store the values accepted by the proxy
  • Validated the incoming value before sending it to the proxy
  • Updated table fields to support only accepted sortable fields.
  • Removed outdated sort function and integrated Grid Proxy client for sorting.

Related Issues

Checklist

  • Tests included
  • Build pass
  • Documentation
  • Code format and docstrings
  • Screenshots/Video attached (needed for UI changes)

- Updated the GridProxy client to include `SortBy` and `SortOrder` fields
- Created a new enum to store the values accepted by the proxy
- Validated the incoming value before sending it to the proxy
- Updated table fields to support only accepted sortable fields.
- Removed outdated sort function and integrated Grid Proxy client for sorting.
@Mahmoud-Emad Mahmoud-Emad marked this pull request as ready for review July 28, 2024 12:49
@Mahmoud-Emad Mahmoud-Emad merged commit ac03039 into development Jul 29, 2024
10 checks passed
@Mahmoud-Emad Mahmoud-Emad deleted the development_contracts_sort branch July 29, 2024 09:05
@xmonader xmonader added this to the 2.6.0 milestone Sep 24, 2024
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.

4 participants