-
-
Notifications
You must be signed in to change notification settings - Fork 61
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
Wrong OneDrive folder #116
Comments
Will have to check up on this, but it's hard for me to replicate, as I don't have a school OneDrive or anything to test with... I'm gonna need some help on this one |
@AlbertMN I can help with testing or even coming up with the fix if needed. It's been like 4 years since I've touched C# though. |
@brij1234 you're very welcome to take a look at the code base, it's been ages since I last looked at the OneDrive folder detection stuff, but should be easy to find in the project. I can assist, but it'd be epic if you'd take a look. |
Looks like the code points to the generic OneDrive environment variable... OneDrive has 3 different EV's %OneDrive% - I think this is the first setup OneDrive account on the computer? |
Nice, didn't know there were more. I guess the software could check whether multiple exists, and in that case, show a Dropdown allowing the user to select which OneDrive to be used. Odd that Will see what I can do, but it'll be some time before I can implement it. |
Thats interesting, would have thought that you'd have the consumer EV on your machine.. I guess it only comes up if you have both personal and commercial accounts configured. For me personally, I have both a personal (consumer) OneDrive and commercial OneDrive.. mainly because I have my own domains, but also useful for work when I want to test new stuff... |
Thanks, will let you know if it'd help, thinking of just faking the variables for now. |
Describe the bug
I selected personal OneDrive account, but my school OneDrive folder is used.
To Reproduce
Steps to reproduce the behavior:
Hardware & software info
Workaround:
Go into advanced settings and manually set the correct folder path.
Additional context & log files
log.txt
.
The text was updated successfully, but these errors were encountered: