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

Add Beataroni info #265

Merged
merged 1 commit into from
Jan 17, 2021
Merged

Add Beataroni info #265

merged 1 commit into from
Jan 17, 2021

Conversation

gary-kim
Copy link
Contributor

Add information about Beataroni.

@geefr

Signed-off-by: Gary Kim <gary@garykim.dev>
@geefr
Copy link

geefr commented Jan 11, 2021

Hey thanks for putting this together.

Info on beataroni is good, I may improve the ui/workflow but the instructions should remain valid for a long time.
(Couldn't get the images to load but assume it'll be right when merged/hosted)

While it's being changed, and assuming the maintainers agree I think these should also be done:

  • Delete, archive, or collapse the section about using a virtual machine. I'm sure it'll work, but wouldn't recommend it
  • In the section about wine it references BeatSaberModManager.exe, that's the old one, so that section could do with some updates
    • I think mod assistant does work under wine, as long as winetricks dotnet is setup, and the dll override is done. Haven't used mod assistant on linux myself though for a long time.

@Caeden117
Copy link
Member

Another thing that could be considered: How about we bring the Linux modding guide out of the Modder's guide and into some part of the Beginner's Guide?

@bloodcloak
Copy link
Member

That was something on my todo list after realizing it was more for playing than making mods on linux.

Still havent gotten to it because it needs an update away from the old mod manager.

Probably will hook onto this pr for that change when reviewing if someone else doesnt get to it before me

@gary-kim
Copy link
Contributor Author

So, in that case, how about we separate the cleanup out into a second PR? I can start working on that once we merge this.

@bloodcloak bloodcloak merged commit aceb305 into bsmg:master Jan 17, 2021
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.

4 participants