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

Wrong OneDrive folder #116

Open
brij1234 opened this issue Oct 12, 2021 · 7 comments
Open

Wrong OneDrive folder #116

brij1234 opened this issue Oct 12, 2021 · 7 comments
Assignees
Labels
bug cant-replicate The developers can't replicate the issue - tough one to fix help wanted

Comments

@brij1234
Copy link

brij1234 commented Oct 12, 2021

Describe the bug
I selected personal OneDrive account, but my school OneDrive folder is used.

To Reproduce
Steps to reproduce the behavior:

  1. Set up a personal and a work OneDrive
  2. go through ACCsetup.

Hardware & software info

  • [Version of AssistantComputerControl] = 1.4.4

Workaround:
Go into advanced settings and manually set the correct folder path.

Additional context & log files
log.txt
.

@AlbertMN
Copy link
Owner

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 AlbertMN added cant-replicate The developers can't replicate the issue - tough one to fix help wanted labels Dec 12, 2021
@brij1234
Copy link
Author

@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.

@AlbertMN
Copy link
Owner

@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.

@pantherale0
Copy link

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?
%OneDriveConsumer% - Points to a personal OneDrive account
%OneDriveCommercial% - Points to the commercial version of OneDrive (loosly based off SharePoint), for all basic purposes, the commercial version refers to school and work accounts.

@AlbertMN
Copy link
Owner

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 %OneDrive% points to your school OneDrive though. I only have consumer OneDrive, but %OneDriveConsumer% isn't a variable I have (nor the commercial one).

Will see what I can do, but it'll be some time before I can implement it.

@pantherale0
Copy link

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...
I'm guessing in that case, %OneDrive% is the default account (the first one setup on the computer) then when you add a commercial account it will create the other EV's.... I think I've got some spare O365 licenses so if you need one to test with, just shout 👍

@AlbertMN
Copy link
Owner

Thanks, will let you know if it'd help, thinking of just faking the variables for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cant-replicate The developers can't replicate the issue - tough one to fix help wanted
Projects
None yet
Development

No branches or pull requests

3 participants