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

User gender is assumed to be male. #480

Closed
geedubess opened this issue Apr 26, 2021 · 2 comments · Fixed by #493
Closed

User gender is assumed to be male. #480

geedubess opened this issue Apr 26, 2021 · 2 comments · Fixed by #493
Assignees
Labels
enhancement New feature or request

Comments

@geedubess
Copy link

During a nostalgic playthrough of Kyrandia, I realized that you can't proceed past Level 14 without having two differently-gendered accounts logged in: https://www.dragonsworn.net/kyrandia.html#walk. Unfortunately, gender appears to be hard-coded in this emulation.

It would be great if there were a way to set the gender associated with a user account in order to have differently gendered accounts active at the same time.

sex = (byte)'M'; //Set everyone to male for now

@enusbaum
Copy link
Member

So there are two ways we can handle this.

  1. Add it to the signup process by adding an additional question for sex/gender during signup
  2. Add a command which allows users to modify this on the fly and have it persist when their record is written to BBSUSR.DAT

I'm leaning 2 as it's more flexible, but 1 would probably be the easiest. Perhaps both?

@enusbaum enusbaum self-assigned this Apr 26, 2021
@enusbaum enusbaum added the enhancement New feature or request label Apr 26, 2021
@geedubess
Copy link
Author

geedubess commented Apr 27, 2021

I am not invested in the implementation, but (2) does seem more flexible, provided the configuration persists and may be modified from a MBBS connection (vs just the console), and would be helpful if there are existing accounts that need to be modified. The signup process is incredibly brief (username/pass/email), so I'm not sure if it's appropriate to start adding more to it.

@tuday2 tuday2 self-assigned this Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants