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

filelist: parse response with STJson #14740

Merged
merged 1 commit into from
Oct 4, 2023
Merged

Conversation

mynameisbogdan
Copy link
Contributor

No description provided.

@garfield69
Copy link
Contributor

garfield69 commented Oct 4, 2023

azure devops services are offline atm so I cannot DL and test.
I'll copy the source and startup my VS2022 and build/test it that way.
[edit] NM, azure back online already.

@mynameisbogdan
Copy link
Contributor Author

I'll copy the source and startup my VS2022 and build/test it that way.

Why copy, you can checkout the branch filelist-stjson since I pushed to main repo.

@garfield69
Copy link
Contributor

Oh, good to know. I'll remember that if it ever happens again ;-)

Copy link
Contributor

@garfield69 garfield69 left a comment

Choose a reason for hiding this comment

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

Oh, wait, I don't have an a/c, and neither does ilike2burnthing.
You apparently do, so I presume you tested to your satisfaction.
If it works for you it's fine by me, I don't care anything about code compliance or the niceties of one json processor over another, that's C# stuff ;-b

@mynameisbogdan
Copy link
Contributor Author

yeah, it works 100%. but keep in mind it's stricter.

But you do have an account... Last activity 14 hours ago. Or it was cadatoiva?

@garfield69
Copy link
Contributor

garfield69 commented Oct 4, 2023

But you do have an account... Last activity 14 hours ago. Or it was cadatoiva?

Not me, or I'd have the indexer loaded on Jackett. My a/c manager has an entry for filelist with the annotation 'invite needed' so I've not ever had the chance to register.

@mynameisbogdan
Copy link
Contributor Author

You can ask cadatoiva for an invite, because I think the staff gave him VIP for being a maintainer of Jackett.

@mynameisbogdan mynameisbogdan merged commit 79d26b3 into master Oct 4, 2023
25 checks passed
@mynameisbogdan mynameisbogdan deleted the filelist-stjson branch October 4, 2023 03:49
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.

None yet

2 participants