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

Add support for multiple Elite:Dangerous accounts #505

Open
Djuwanna opened this issue Apr 28, 2018 · 7 comments
Open

Add support for multiple Elite:Dangerous accounts #505

Djuwanna opened this issue Apr 28, 2018 · 7 comments
Labels
9. enhancement The behaviour is as specified, but we would like to modify or extend the spec. significant work Just sayin'

Comments

@Djuwanna
Copy link

Djuwanna commented Apr 28, 2018

Request EDDI be able to discern which Elite Dangerous account is logged in and provide separate configurations for multiple accounts (support for EDSM, Materials, etc.)

Right now, when I switch between multiple Elite accounts I have to remember to reconfig EDDI for the correct account before I log in or I end up with inaccurate information both locally and being uploaded to EDSM (I actually had to have EDSM delete one of my accounts so I could recreate it and reupload the correct information - my fault obviously, but multiple account support would have prevented my error). And I have to replace the materials .json file with a "zeroed out quantities" materials .json file when switching accounts or the only things that are updated are those materials I have, not those I don't have (which I'm betting is a Frontier thing - they probably don't send zero quantity information).

EDMC (as an example) is able to switch accounts on-the-fly and remembers different logons and API's for the various accounts. Can EDDI be written at some point to do the same?

@Tkael Tkael added the 9. enhancement The behaviour is as specified, but we would like to modify or extend the spec. label May 4, 2018
@richardbuckle richardbuckle added the needs thought For when you need to slow down and consider things. label Jun 28, 2018
@Tkael
Copy link
Member

Tkael commented Nov 3, 2018

Also, adding this would allow us to support having different configurations for live and beta releases.

@Tkael
Copy link
Member

Tkael commented Dec 2, 2018

There is an unimplemented Commander event that we can use to ensure all commander data is recorded under the correct commander name during the initial loading journal events (the old Commander continued event fired too late to be reliable).

Initial thought would be to add the commander name to the configuration data files saved to %appdata%/EDDI by renaming the files. E.g. shipmonitor.json could become (name).shipmonitor.json or (name).beta.shipmonitor.json
What character set does Elite Dangerous allow for commander names? With that sort of formatting, the commander name must be able to be saved as a valid file name.

The other option would be to include multiple cmdr names in a single json file.

@mwerle
Copy link

mwerle commented Jun 2, 2019

The other option would be to include multiple cmdr names in a single json file.

Probably safest, generate a uuid (or calculate a hash) over the Commander name and use that elsewhere; avoids any potential characterset and formatting issues.

@Timnus
Copy link

Timnus commented Apr 6, 2020

as far as i know. Frontier allows the full qwerty keyboard character set for commander names. there is a Character length limit.

@Tkael
Copy link
Member

Tkael commented Jun 5, 2020

When we implement this, we can also implement storing the active config in memory to reduce unnecessary reads from the file system.

@Tkael
Copy link
Member

Tkael commented Jun 9, 2023

@parduz has suggested having EDDI be as portable as possible configuring the location of EDDI's settings during installation (perhaps via a settings file saved with the .exe?)

@juhaeske
Copy link

juhaeske commented Apr 2, 2024

Any possibility/work around to add multiple cmdr feature?
And I do not mean multiple windows accounts. EDDiscovery has it, EDEngineer too.
3 Cmdrs hete...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9. enhancement The behaviour is as specified, but we would like to modify or extend the spec. significant work Just sayin'
Projects
None yet
Development

No branches or pull requests

6 participants