-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat(mobile): search enhancement #8392
Conversation
…ich into mobile/search-enhancement
Deploying immich with
|
Latest commit: |
91baeac
|
Status: | ✅ Deploy successful! |
Preview URL: | https://0c9e8fed.immich.pages.dev |
Branch Preview URL: | https://mobile-search-enhancement.immich.pages.dev |
}; | ||
} | ||
|
||
factory SearchLocationFilter.fromMap(Map<String, dynamic> map) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we use/need to/fromMap and to/fromJson at all for the filter classes?
I think they are never used.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, they are not used. It is auto-generated. I use this extension on vscode to help generate these classes
https://marketplace.visualstudio.com/items?itemName=dotup.dart-data-class-generator
This PR adds the advanced search filters feature to the mobile app.
The result is fetched in batches of 1000, the infinite scrolling is janky due to the way the asset grid is built, so when a new batch is fetched, it rerenders the whole page. I might open a subsequent PR to address this issue
searchmobiledemo.mp4