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.

Sign up for GitHub

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

Jump to bottom

[VixenBridge] New bridge #2763

Merged
merged 1 commit into from Jun 9, 2022
Merged

[VixenBridge] New bridge #2763

merged 1 commit into from Jun 9, 2022

Conversation

ghost
Copy link

@ghost ghost commented May 31, 2022 •
edited by ghost
Loading

New bridge for Vixen Network sites. Beware of NSFW content.

@github-actions
Copy link

github-actions bot commented May 31, 2022 •
edited
Loading

Pull request artifacts

file last change
Vixen-pr-context1 2022-06-08, 07:59:34

@dvikan
Copy link
Contributor

dvikan commented Jun 4, 2022

Rename to VixenBridge. Include it in the description so that it's searchable. Move collectData to the top. Use array_column for plucking values out of an array. Inline silly one-liner methods such as getVideosURI. Throw exception instead of calling returnClientError().

@ghost ghost changed the title [VxnBridge] New bridge [VixenBridge] New bridge Jun 8, 2022
@dvikan dvikan merged commit 1b0a6f2 into RSS-Bridge:master Jun 9, 2022
Kwbmm pushed a commit to Kwbmm/rss-bridge that referenced this pull request Jun 17, 2022
@Kwbmm
[VixenBridge] New bridge (RSS-Bridge#2763)
1766905
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dvikan