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 quick start guide #2198

Merged
merged 8 commits into from Jun 11, 2023
Merged

Update quick start guide #2198

merged 8 commits into from Jun 11, 2023

Conversation

ghost
Copy link

@ghost ghost commented Jun 9, 2023

Simpler instructions with Cultivation. Thanks to Nazrin for writing.

Co-Authored-By: Nazrin <1877986+scooterboo@users.noreply.github.com>
@reflexit
Copy link

reflexit commented Jun 9, 2023

Should we keep both Cultivation method (recommended) and manual method (for advanced users)? Because not every user downloads Cultivation, some (including me) prefer configuring things manually.

@ghost
Copy link
Author

ghost commented Jun 9, 2023

Sure, I don't see why not. Feel free to propose something for the manual method and I'll put it there + add you as a co-author to the commit.

@reflexit
Copy link

reflexit commented Jun 9, 2023

The manual method is basically the Quick setup guide part in the current README. Perhaps we can clean it up a little bit and remove some obsolete information.

As a reference, my setup steps are:

  1. Install java, mongodb, and a proxy daemon (I use mitmproxy).
  2. Get grasscutter.jar and resources folder.
  3. Start mongodb service if not already. On Windows the command is net start mongodb.
  4. Run Grasscutter with java -jar grasscutter.jar.
  5. Run proxy daemon and set system network proxy (see Connecting with the client part).
  6. Install a certain client patch. Run the game as administrator (very important).

I've been following these steps since 3.3, and it still works now.

@KingRainbow44
Copy link
Member

the word Genshin is a big no-no

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@ghost
Copy link
Author

ghost commented Jun 9, 2023

the word Genshin is a big no-no

Uh, why?

Just mentioning the game by name doesn't infringe any intellectual "property" rights. It seems simpler to name it.

Also, Genshin is a Buddhist monk, not a game.

falseuniversefacts and others added 7 commits June 9, 2023 19:43
Co-authored-by: Magix <27646710+KingRainbow44@users.noreply.github.com>
Co-authored-by: Magix <27646710+KingRainbow44@users.noreply.github.com>
Co-authored-by: Magix <27646710+KingRainbow44@users.noreply.github.com>
Co-authored-by: Magix <27646710+KingRainbow44@users.noreply.github.com>
Co-authored-by: Magix <27646710+KingRainbow44@users.noreply.github.com>
Co-authored-by: Magix <27646710+KingRainbow44@users.noreply.github.com>
Co-authored-by: Magix <27646710+KingRainbow44@users.noreply.github.com>
@KingRainbow44 KingRainbow44 merged commit 944db42 into Grasscutters:development Jun 11, 2023
@ghost ghost deleted the patch-1 branch June 11, 2023 03:34
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