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

fix: fix sorting and paging not working on config changes #2358

Merged
merged 2 commits into from
Oct 22, 2024

Conversation

mainawycliffe
Copy link
Contributor

No description provided.

Copy link

vercel bot commented Oct 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
aws-preview ✅ Ready (Inspect) Visit Preview Oct 22, 2024 6:46am
flanksource-ui ✅ Ready (Inspect) Visit Preview Oct 22, 2024 6:46am

Copy link

netlify bot commented Oct 18, 2024

Deploy Preview for goofy-euclid-75956c ready!

Name Link
🔨 Latest commit c9cef50
🔍 Latest deploy log https://app.netlify.com/sites/goofy-euclid-75956c/deploys/671747b567974300085881c0
😎 Deploy Preview https://deploy-preview-2358--goofy-euclid-75956c.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.

Copy link

netlify bot commented Oct 18, 2024

Deploy Preview for clerk-saas-ui ready!

Name Link
🔨 Latest commit c9cef50
🔍 Latest deploy log https://app.netlify.com/sites/clerk-saas-ui/deploys/671747b5abfd41000897142c
😎 Deploy Preview https://deploy-preview-2358--clerk-saas-ui.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.

Copy link

netlify bot commented Oct 18, 2024

Deploy Preview for flanksource-demo-stable ready!

Name Link
🔨 Latest commit c9cef50
🔍 Latest deploy log https://app.netlify.com/sites/flanksource-demo-stable/deploys/671747b5afd4fa0008c96066
😎 Deploy Preview https://deploy-preview-2358--flanksource-demo-stable.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.

@moshloop
Copy link
Member

@mainawycliffe it doesn't look like the page size is included in the first request

@moshloop
Copy link
Member

@mainawycliffe can we default the sort on changes to be by last seen ?

@mainawycliffe
Copy link
Contributor Author

@moshloop the backend only allows name, change_type, summary, source, created_at, count for sorting

@moshloop
Copy link
Member

@mainawycliffe - created_at is last seen - before it was sorted by most recent, and now its unsorted which doesn't make sense when looking at history

@mainawycliffe mainawycliffe force-pushed the fix-sorting-changes-not-working branch from 6fdce32 to c9cef50 Compare October 22, 2024 06:35
@moshloop moshloop merged commit 2a22ce0 into main Oct 22, 2024
15 of 17 checks passed
@moshloop moshloop deleted the fix-sorting-changes-not-working branch October 22, 2024 08:58
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.

2 participants