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

[Various] getKey replacements and docu #3283

Merged
merged 4 commits into from
Mar 6, 2023

Conversation

Bockiii
Copy link
Contributor

@Bockiii Bockiii commented Mar 6, 2023

Added the new getKey function to the documentation and replaced instances of custom functions with the new global.

9gag and Brut will fail because they bridges are broken, but that has nothing to do with the getkey :)

@github-actions
Copy link

github-actions bot commented Mar 6, 2023

Pull request artifacts

file last change
AllocineFRBridge-current-context1 2023-03-06, 13:24:44
AnimeUltimeBridge-current-context1 2023-03-06, 13:24:44
BandcampDailyBridge-current-context1 2023-03-06, 13:24:44
BandcampDailyBridge-current-context2 2023-03-06, 13:24:44
BandcampDailyBridge-current-context3 2023-03-06, 13:24:44
BandcampDailyBridge-current-context4 2023-03-06, 13:24:44
BrutBridge-current-context1 2023-03-06, 13:24:44
GBAtempBridge-current-context1 2023-03-06, 13:24:44
GithubTrendingBridge-current-context1 2023-03-06, 13:24:44
InternetArchiveBridge-current-context1 2023-03-06, 13:24:44
NineGagBridge-current-context1 2023-03-06, 13:24:44
NineGagBridge-current-context2 2023-03-06, 13:24:44
NpciBridge-current-context1 2023-03-06, 13:24:44
PixivBridge-current-context1 2023-03-06, 13:24:44
PixivBridge-current-context2 2023-03-06, 13:24:44
SplCenterBridge-current-context1 2023-03-06, 13:24:44
TwitScoopBridge-current-context1 2023-03-06, 13:24:44
UnogsBridge-current-context1 2023-03-06, 13:24:44
UnogsBridge-current-context2 2023-03-06, 13:24:44
WebfailBridge-current-context1 2023-03-06, 13:24:44
AllocineFRBridge-pr-context1 2023-03-06, 13:24:44
AnimeUltimeBridge-pr-context1 2023-03-06, 13:24:44
BandcampDailyBridge-pr-context1 2023-03-06, 13:24:44
BandcampDailyBridge-pr-context2 2023-03-06, 13:24:44
BandcampDailyBridge-pr-context3 2023-03-06, 13:24:44
BandcampDailyBridge-pr-context4 2023-03-06, 13:24:44
BrutBridge-pr-context1 2023-03-06, 13:24:44
GBAtempBridge-pr-context1 2023-03-06, 13:24:44
GithubTrendingBridge-pr-context1 2023-03-06, 13:24:44
InternetArchiveBridge-pr-context1 2023-03-06, 13:24:44
NineGagBridge-pr-context1 2023-03-06, 13:24:44
NineGagBridge-pr-context2 2023-03-06, 13:24:44
NpciBridge-pr-context1 2023-03-06, 13:24:44
PixivBridge-pr-context1 2023-03-06, 13:24:44
PixivBridge-pr-context2 2023-03-06, 13:24:44
SplCenterBridge-pr-context1 2023-03-06, 13:24:44
TwitScoopBridge-pr-context1 2023-03-06, 13:24:44
UnogsBridge-pr-context1 2023-03-06, 13:24:44
UnogsBridge-pr-context2 2023-03-06, 13:24:44
WebfailBridge-pr-context1 2023-03-06, 13:24:44

@Bockiii
Copy link
Contributor Author

Bockiii commented Mar 6, 2023

Dont merge yet, I found some additional bridges

@Bockiii
Copy link
Contributor Author

Bockiii commented Mar 6, 2023

The global function was missing a way to handle mixed content of global values and context values. Didn't notice it during my testing, but the pixiv bridge brought it up. All previous changes should still work and the new ones are tested as well.

@Bockiii Bockiii marked this pull request as draft March 6, 2023 13:17
@Bockiii Bockiii marked this pull request as ready for review March 6, 2023 13:29
@dvikan dvikan merged commit f0e5ef0 into RSS-Bridge:master Mar 6, 2023
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.

2 participants