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

Create 4 character slots per account #307

Closed
wants to merge 1 commit into from

Conversation

Arkblade
Copy link

Allow each account to have 4 character slots. Reduces amount of accounts needed, encourages play from vets and new players alike. Note: 103 has already adopted this change.

Allow each account to have 4 character slots. Reduces amount of accounts needed, encourages play from vets and new players alike.
@Nagath
Copy link
Contributor

Nagath commented Jan 24, 2015

If I remember right, on 103 they added the slots to all existing accounts with a comandline in the admin shell and then changed the script on the website for account creation.

@Koveras
Copy link
Contributor

Koveras commented Feb 1, 2015

+1
And +2 slots for all existing accounts sounds good too :)

@Arkblade
Copy link
Author

Arkblade commented Feb 2, 2015

Heh I thought it was clear when I said "Allow each account to have 4 character slots" that it meant ALL accounts, including old accounts. My mistake then, I mean ALL accounts; past present and future. =)

cyberjunk pushed a commit to cyberjunk/meridian59-orig that referenced this pull request Jan 13, 2017
Mail and wall offset fixes, increment client version.
@davidklaw
Copy link
Contributor

I'm reviewing old PRs looking for cleanup opportunities and I know that this is being actively discussed. Given that, the age of this PR, and the lack of any code changes, I recommend closing this PR ☠️

@akirmse
Copy link
Collaborator

akirmse commented Jun 21, 2022

There doesn't seem to be any code in this request, either.

@akirmse akirmse closed this Jun 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants