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

Server v2 #167

Draft
wants to merge 153 commits into
base: cuttingedge
Choose a base branch
from
Draft

Server v2 #167

wants to merge 153 commits into from

Conversation

C9Glax
Copy link
Owner

@C9Glax C9Glax commented Apr 19, 2024

@db-2001
While going over your PR (related issue) I thought about some off the issues we have with the API.
#113 (comment)

make the /Jobs/Waiting smaller? It seems like it's sending 3.8MB of text every second, which is hell on my data usage when accessing remotely. Is there a need to sync the full metadata of all manga jobs every second, instead of on demand?

Resolves #74
Resolves #114
Could be their own Requests.

By returning a list of JobIds and crosschecking locally which jobs exist, this should be a lot easier on data.

Generally the API transmits a lot more data than it needs to, so streamlining requests (stripping data, splitting single requests into multiple smaller ones) we can react a lot better to changes.

So extending your effort, I propose that we also get going on a Version 2 of the API.
The old API would still be accessible (for now), but we could gradually shift requests to the new one.

I have started writing out the API-Documentation.
If you have any more requests you would like to see, then write them in this PR.

@C9Glax C9Glax added the enhancement New feature or request label Apr 19, 2024
@db-2001
Copy link
Collaborator

db-2001 commented Apr 19, 2024

Yeah this makes sense, let's make a branch of the current working code, like a V01.XX.XX and generate a docker image for that so people can force their server to run that code.

@db-2001
Copy link
Collaborator

db-2001 commented Apr 19, 2024

And you can just merge my current json API branch into this Server V2 branch if you want a head start. Once that's done we can close my branch and Implement json API as part of V2 and not have any in V1

@C9Glax
Copy link
Owner Author

C9Glax commented Apr 19, 2024

You want a hard cut-over? The way I wrote the current requests, we could still have both parts running, and be future-proof (v3,4,5...) too.
But yea I will merge your current branch into this one

@db-2001
Copy link
Collaborator

db-2001 commented Apr 19, 2024

I think something as big as an API changing structure would be enough to do a hard cut-over, it's your call in the end but I think V2 being the version with the big API change makes sense to me and then we just leave the old version alone, maybe patch it every now and then if we need to but otherwise leave it alone.

@db-2001
Copy link
Collaborator

db-2001 commented Apr 20, 2024

Also do you want to take the lead on the server side of V2 then? That'll let me focus on doing the frontend V2 implementing all these manga pop-up and search/add pop-up changes that everybody's been wanting. I can also then implement any other changes as a result of the server V2. What do you think?

@C9Glax
Copy link
Owner Author

C9Glax commented Apr 20, 2024

Sounds like plan!
So it would essentially be complete V2 front- and backend, then a hard cut-over would make a lot of sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment