💥 SeaORM + GraphQL migration #785
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* wip: start macro for filter gen * lowkey hate macros * restore meta filter * show example macro not working * no idea but it works now * woe * :( * note about migration * cleanup * debug * fix filters, add data loaders for read sessions * rm debugs
…s' into breaking/sea-orm
- Fix for old prisma logic to work for new epub-rs - Fix for updating reading session - Adding unit tests for malformed epub files
…aking/sea-orm-entity-unit-tests
* moving library_hidden_to_user query to the entity
…aking/sea-orm-more-changes
* wip: start rest api port * wip: port auth * wip: auth helpers * wip: auth middlewares * wip: auth middlewares and api key handling * wip: remove handrolled entities in core * wip: fix opds v2 * add notes * wip no compile * wip: koreader port * update device id if changed * comment out things, port opds v2 search * port browse_libraries * port browse_library_by_id * library by id catalogs * opds v2 ported but not very tested * opds v1.2 port * fix bad query * port thumbnail api * update todo, fix clippy lint * remove utoipa * add middleware, add page getter
Attempting to resolve some really funky Pressable issues with expo-router
…es (#802) * native switches for all ios versions * fix missing thumbnail card misalignment * allow "mark as read" on subsequent readings
I did a little redesigning of the sheet content, taking some of the transparent header logic from @Arklaum via Arklaum@bbf71b2
Relates to #799
The conflict clause just needed the column to be added
There are changes specifically around management/cleanup of tasks, hoping this helps with memory pressure or crashes a couple folks reported
I don't have time to debug the actual backfill operation so just resolved to using the set of follow-up fixes referenced by a user debugging the issue #799 (comment)
* ✨ Toggle needing ctrl / cmd for panzoom * fix rust check
* ✨ Basic browser thumbnail ratio selector * minor reading time formatting changes + browser locale support * fix setPageSize * style like the expo footer + fix dynamic height issues * update remaining thumbnail ratios * remove custom List component from ReaderFooter * account for thumbnail ratio on home page * update graphql schema * fix smooth scrolling on reader footer * no smooth scrolling when changing reading direction * fix landscape check * update comment * minor formatting changes
This is a short-term fix. A proper fix should actually be explored
Moving forward, unstable images (if created) will come from an `unstable` branch
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See #634
This was effectively a rewrite of Stump to fully migrate off of Prisma, after the Rust client unmaintained status was announced, and swapping to a GraphQL API from REST. As expected with this scope of a change, there are many breaking changes present.
This PR will stay in draft until I complete the following:
I'm also unsure how I want this to be merged. I'm thinking I'll either flatten the very early set of commits with chaotic names and retain the history or just flatten it all
Thanks to everyone who contributed and helped to speed this slog of a task up! Expect lots of bugs once this lands in
experimental