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

Use Fallback Vault Location as Default #1848

Closed
Sheikah45 opened this issue Aug 3, 2020 · 4 comments · Fixed by #1995
Closed

Use Fallback Vault Location as Default #1848

Sheikah45 opened this issue Aug 3, 2020 · 4 comments · Fixed by #1995
Milestone

Comments

@Sheikah45
Copy link
Member

Currently the client tries to place maps and mods into a sub folder in the documents folder. This causes issues with some players not having maps be downloaded correctly or having a difference between where the game looks for maps and where maps are downloaded.

This could potentially be fixed by defaulting to the fallback vault location in programdata as the default and also gives a consistent place to find the maps

Will need to include ability to either migrate maps or use both locations for compatibility

@foodlfg
Copy link

foodlfg commented Aug 20, 2020

especially on Linux.
you have to turn it on anyways if you want a working client.
so it makes sense.

i dont know what is the disadvantage of this on windows.

@1-alex98
Copy link
Member

Well there is code that turn it on on linux

@foodlfg
Copy link

foodlfg commented Aug 20, 2020

Well there is code that turn it on on linux

oh nice this means i can remove that step from the wiki.

@1-alex98
Copy link
Member

1-alex98 commented Aug 20, 2020

Better try before removing it but should happen automatically

1-alex98 pushed a commit that referenced this issue Oct 18, 2020
@1-alex98 1-alex98 added this to the v1.2.2 milestone Oct 18, 2020
Sheikah45 added a commit that referenced this issue Nov 6, 2020
mrchris2000 pushed a commit to mrchris2000/downlords-faf-client that referenced this issue Apr 15, 2022
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 a pull request may close this issue.

3 participants