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

Update GameManager.cs #45 #46

Closed

Conversation

mikebarrett19
Copy link

see #45

@Snailedlt Snailedlt requested a review from Mikou27 January 9, 2023 18:04
@Mikou27
Copy link
Collaborator

Mikou27 commented Jan 11, 2023

Hi, that's very interesting and it's among the most requested features/changes. I'm not familiar with assemblies for now, so i will have to take the time ro review it correctly. Maybe we could get in touch if you're interested at contributing more to the project? EDIT: I forgot to ask you if you have finally got it to work the way you want with Playnite.

@Snailedlt
Copy link
Member

@mikebarrett19 did you get it working with playnite? :)

@mikebarrett19
Copy link
Author

mikebarrett19 commented Feb 19, 2023

@Mikou27 @Snailedlt Sorry y'all, work has been busy and your messages got lost. I haven't gotten it to work with playnite, it wasn't as "drop in" as I thought it would be, as so much of the configuration is based in the gui. I plan to take another crack at it one of these days, I'll probably need to contribute default controller to screen mappings or something. It's up to you guys what to do with this pull request. We could close it until I get something working in playnite. I doubt I'll get to it until at least this summer. I'm not a C# developer by trade, but I plan on using contributions to this and playnite to learn C# in my free time. I need the assembly source changed because when I run nucleus dlls from another application (playnite) the directories are incorrect because they're sourced from the main application. But since I'm new to C# and windows development in general I have no idea the implications of this change, just that it fixes the directory problem. I can look into other fixes if this causes some issues. Appreciate y'all!

@Snailedlt
Copy link
Member

I'm not actively working on the project, so @Mikou27 should decide what to do with the PR :)

@Mikou27 Mikou27 closed this Jun 13, 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.

3 participants