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

[Next.Ink] Bridge rewrite (#3863) #3902

Closed
wants to merge 1 commit into from
Closed

[Next.Ink] Bridge rewrite (#3863) #3902

wants to merge 1 commit into from

Conversation

ORelio
Copy link
Contributor

@ORelio ORelio commented Jan 19, 2024

Reimplement NexINpact bridge, now named Next.Ink bridge. Fixes #3863.

Note that due to major changes on the website, I had to change some parameters so the new bridge is not compatible with existing feeds and they need to be recreated. I took this opportunity to rename the file.

Reimplement NexINpact bridge, now named Next.Ink bridge
Copy link

Pull request artifacts

Bridge Context Status
NextInk 1 untitled (pr) ⚠️ INFO Bridge not found: NextInpact
⚠️ (shutdown) 8192: Creation of dynamic property NextInkBridge::$briefFilter is deprecated in bridges/NextInkBridge.php line 67
NextInpact 1 untitled (current) Bridge returned error 404! (19741)
Type: HttpException
Message: https://www.nextinpact.com/rss/news.xml resulted in 404 Not Found <!DOCTYPE html>
⚠️ INFO Bridge not found: NextInk

last change: Friday 2024-01-19 19:07:27

@ORelio
Copy link
Contributor Author

ORelio commented Jan 19, 2024

There is an issue with the diff.

@ORelio ORelio closed this Jan 19, 2024
@ORelio
Copy link
Contributor Author

ORelio commented Jan 19, 2024

I have an issue with my fork and will reopen a proper pull request once my other pull requests get merged.

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.

NextInpact Bridge failed with error 404
1 participant