Skip to content

Wrong OneDrive folder #116

Open
Open

Description

@brij1234

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
.

Activity

  1. AlbertMN commented on Dec 12, 2021

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

  2. brij1234 commented on Dec 12, 2021

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

  3. AlbertMN commented on Dec 12, 2021

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

  4. pantherale0 commented on Dec 15, 2021

    @pantherale0

    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.

  5. AlbertMN commented on Dec 16, 2021

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

  6. pantherale0 commented on Dec 16, 2021

    @pantherale0

    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 👍

  7. AlbertMN commented on Dec 16, 2021

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

Metadata

Metadata

Assignees

Labels

bugcant-replicateThe developers can't replicate the issue - tough one to fixhelp wanted

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions