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

Added trading support #227

Open
wants to merge 26 commits into
base: master
Choose a base branch
from

Conversation

itzandroidtab
Copy link
Collaborator

@itzandroidtab itzandroidtab commented May 16, 2023

Briefly describe what changes this PR introduces

Added trader support. Allows 2 users to trade items with eachother.

Support for:

  • registering as a trader (lv 5 and 25 gold as the requirements)
  • Multiple trade channels
  • Chat in channels
  • Chat during trade
  • Cleanup when a user crashes in a trade
  • 15 Gold trade fee
  • Trading items

Basic support for:

  • Putting a item in a empty slot. For now the server assumes every item only uses 1 slot so there might be overlapping items after a trade

Link to all issues that this PR solves

#199
#191

Relevant pictures/videos of the PR (optional)

Dark.and.Darker.2023.05.16.-.23.31.14.01.mp4

Checklist

  • This change introduces new dependencies and I have included updated pyproject.toml and poetry.lock files.
  • This change requires database migrations and I have included an alembic migration.

@itzandroidtab itzandroidtab changed the title Added trader support Added trading support May 16, 2023
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.

3 participants