Skip to content

Conversation

@annavik
Copy link
Member

@annavik annavik commented Sep 25, 2024

Fixes #594

Summary of changes:

  • Add overview tabs to URL params
  • Add polling to collection endpoints
  • Update population status logic (show button as loading while the collection is being populated)
  • Show correct label in "Created at" column for collections

@annavik annavik requested a review from mihow September 25, 2024 07:56
@netlify
Copy link

netlify bot commented Sep 25, 2024

Deploy Preview for ami-dev ready!

Name Link
🔨 Latest commit d4d7353
🔍 Latest deploy log https://app.netlify.com/sites/ami-dev/deploys/66f3c227bd250c00084db378
😎 Deploy Preview https://deploy-preview-595--ami-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 31
Accessibility: 89
Best Practices: 92
SEO: 100
PWA: 80
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Sep 25, 2024

Deploy Preview for ami-storybook ready!

Name Link
🔨 Latest commit d4d7353
🔍 Latest deploy log https://app.netlify.com/sites/ami-storybook/deploys/66f3c227faab130008061f01
😎 Deploy Preview https://deploy-preview-595--ami-storybook.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.

import { UserPermission } from 'utils/user/types'
import { useAuthorizedQuery } from '../auth/useAuthorizedQuery'

const REFETCH_INTERVAL = 10000 // Refetch every 10 second
Copy link
Collaborator

Choose a reason for hiding this comment

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

Isn't this constant already somewhere else? For the table views?

Copy link
Collaborator

@mihow mihow left a comment

Choose a reason for hiding this comment

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

I left one comment, but merging now because its a great improvement

@mihow mihow merged commit 3cd61e1 into main Sep 26, 2024
6 checks passed
@mihow mihow deleted the feature/overview-tabs-params branch September 26, 2024 02:57
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.

Add project overview tabs to URL params & auto-refresh

3 participants