-
Notifications
You must be signed in to change notification settings - Fork 149
Update Junk-Store 1.1.12 #938
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
base: main
Are you sure you want to change the base?
Conversation
… for future server side filtering of the RSS feed.
Issues FoundNo issues with your PR description were found. Next Steps
Thank you for your contribution! If you need any help, please reach out on our Discord server. ❤️ |
EMERALD0874
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for testing
|
@EMERALD0874 is there something I need to do before this gets approved for deployment? We'd like to get on top of the testing tasks soon as we'll be travelling and that will mess with the timelines of the PR workflow. |
|
It's been deployed now, sorry. Thought I got it already but may have missed it |
|
Thank you, no worries. I just thought I missed something on my end. |
Plugin Testing ReportInstalled Plugins
Specifications
IssuesHas the following major blocking issue(s):
Has the following minor non-blocking issue(s):
SummaryOpened the plugin and installed dependencies. Hit the Games button and was taken to a view with an Epic tab. Tried to use the PlayStation and Nintendo login buttons and was shown a dialog that said it had opened in a browser window, but the Steam menu shows no other apps or views open. Went to desktop mode and used the login app, which succeeded in opening a Firefox tab where I could log in to PlayStation network and the Epic login was successful. Returned to Game Mode and launched the Epic view. It didn't detect games already installed from a previous Epic install but that's probably expected. Kicked off a game install and that seems to be working fine. |
|
@wheaney thanks for the test report. The blockers are not actually plugin related issues. Legendary is an external application. The oauth login issue with browsers is actually a gamescope limitation and nothing I can do about it, I had off control to steam for the login process so I'm not even involved in it and legendary does the work there. as for the non blockers, the FOSS version only has Epic support. Gog is a paid extension and not part of this code base. The other extensions you might have seen is part of the commercial product that's a completely separate code base and not relevant here. The non printable characters has been there since the original release, it doesn't affect functionality and most users only do this once and never look at it again. I have found that sometimes these can be font related but honestly implementing a whole lot of stuff for just this is not really a priority and has never stopped merging before. Only detecting games managed by the plugin is kinda the point. I would how ever appreciate feedback on the Umu changes I made, it seems that it works properly for games now that doesn't have an UMU id, which is what this whole PR was actually about. But we have identified that there's an issue with UMU fixes for games that do have an ID. I haven't had time to sit down and investigate it yet due to it being that time of year. I'll be returning to my boat in the next few days so I'll probably have some time to look at it once I'm back in work mode. If you could ID some games that exhibit this behaviour as test cases that would be tremendously helpful. |
Okay, it seemed worth calling out as I was looking at the Junk Store web page and it advertises that you never need to leave game mode, but for this you do, at least as a first-time setup. And maybe if the login expires?
Can you help me figure out what games qualify? I'm not familiar with this issue. |
This is actually in the troubleshooting area of the website. It's well documented. If you don't use oauth you never leave game mode and most users use email and password. It's also not a plugin feature, this is a legendary issue (or any oauth/webview implementation).
This is actually the crux of the issue. When you launch a game the first time if you get a "path not found" error, it's one that qualifies. you might be able to trawl the umu fixes db for this... but it's super time consuming to try and match ID's with the games you own. We just install games until we find one that's an issue. You can use the games tested table on the website to see the ground we have already covered. |
|
Dying Light: Enhanced Edition works perfectly, that's not listed in the tested table. |
|
Elite Dangerous also worked well for me, after fighting with its launcher, that's also not in the tested list. |

… for future server side filtering of the RSS feed.
Update Junk-Store to 1.1.12
Fixed a bug with umu fixes causing a path not found error.
Task Checklist
Developer
Plugin
Backend
Community
Testing