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

Add 1ptbar index and modify soulvoice login to cookie #9596

Merged
merged 14 commits into from
Sep 22, 2020
Merged

Add 1ptbar index and modify soulvoice login to cookie #9596

merged 14 commits into from
Sep 22, 2020

Conversation

rxrw
Copy link

@rxrw rxrw commented Sep 22, 2020

I'm coming again!

@jackettbot
Copy link

jackettbot bot commented Sep 22, 2020

Hi @reprover,

Thanks for your contribution to Jackett!

If you are adding a new indexer, please ensure that you've added it to the readme as well

A human will be along soon to review

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.

In future, when adding a new site, please raise a separate PR
in the event we have to back out your soulvoice change, we end up having to also back out 1ptbar.
having separate PR allows better management of changes.

src/Jackett.Common/Definitions/1ptbar.yml Outdated Show resolved Hide resolved
src/Jackett.Common/Definitions/1ptbar.yml Show resolved Hide resolved
@garfield69 garfield69 linked an issue Sep 22, 2020 that may be closed by this pull request
@rxrw
Copy link
Author

rxrw commented Sep 22, 2020

I'm sorry for my error. I'll create a new branch tomorrow since there's very late in my location.
btw, I'm sure it works well beacuse I'm using it :-)

@garfield69 garfield69 merged commit 7babbff into Jackett:master Sep 22, 2020
garfield69 added a commit that referenced this pull request Sep 22, 2020
@garfield69
Copy link
Contributor

Jackett 0.16.1327

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.

[req] 1ptbar
3 participants